Skip to content

Update - Allow user to edit their submitted photos#532

Open
xlcrr wants to merge 44 commits intostagingfrom
photos/view-old
Open

Update - Allow user to edit their submitted photos#532
xlcrr wants to merge 44 commits intostagingfrom
photos/view-old

Conversation

@xlcrr
Copy link
Copy Markdown
Member

@xlcrr xlcrr commented Sep 18, 2022

In this PR we want to give users the ability to edit photos they have already submitted.

Trello Card

Screenshot 2022-10-29 at 17 41 55

During BulkTag.vue, there is a new option to filter your photos by Not Submitted (photo.verified = 0) and Verified (photo.verified = 2).

Photos that are in review are not editable yet. Only ones that have been verified are.

Depending on the status that is selected, when the user submits the tags, it will trigger either AddTagsToPhoto action or UpdateTagsOnPhoto action on the backend via UserPhotoController@bulkTag

With the original AddTagsToPhoto request, we used the existing tags, custom tags, and brands, to calculate various XP scores.

Before we do anything, we use the existing tags to reverse most of these jobs.

We then delete the existing tags, and re-create the Photo with the new tags, updating everything again from scratch in the process.

Maybe there is a better way to do this. Would like 2nd opinion before moving forward.

@xlcrr xlcrr added the help wanted Extra attention is needed label Oct 29, 2022
@xlcrr xlcrr requested a review from GeniJaho October 29, 2022 16:40
@GeniJaho
Copy link
Copy Markdown
Collaborator

PR looks good to me Sean, I think you really want to add some tests for the main features you're introducing. I'm sure you'll find a bug or two while adding tests 😅
You could test the DB integrity and everything that might be affected by this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants