Requested fixes for Ian, 6/28#74
Merged
Merged
Conversation
…g marking/cover contributions to be updated before editor review.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
7f9818e- Restore contributor edit actions for pending submissionsRestores the ability for contributors to edit their pending marking and cover submissions before editor review. The backend now accepts updates to draft, pending, needs-revision, and rejected contribution rows, while keeping non-draft edits pending for review. It also adds contribution consolidation so older non-draft submissions for the same contributor and catalog target are superseded, deleted, and recorded through audit transactions. The dashboard and contribution detail pages now route pending, returned, approved, and cover submissions to the correct edit flows.
167013b- Deduplicate manual v1 color-split auxmark rowsAdds v1 import deduplication for rows that only differ by manual color-split fields. The munger now drops later duplicate rows when parser-relevant fields are otherwise identical, while preserving rows that have attached images so image lineage remains valid. Test coverage was added for duplicate color-split rows, including cases where image-bearing duplicates must be retained.
49e7a39- Add editor image quick actions and fix relationship inheritanceAdds editor-facing image management controls on marking and cover detail pages. Staff can now set the default catalog image directly from the main gallery or thumbnail strip, reorder thumbnails, and delete image rows without going through the full edit form. The commit also improves v1/munger relationship handling by preserving multi-star inscription text, resolving Same entries from the immediate previous sibling instead of the top parent, and carrying that inheritance into v1 overlay behavior. Tests cover the star-preservation and inheritance cases.