Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MBS-8500: Add "not by me" filter for collection edit lists #3453

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

reosarevok
Copy link
Member

Implement MBS-8500

Problem

Collection edit lists cannot be filtered, and it can be very hard to find others' edits to review on a collection where you are fairly active yourself as an editor.

Solution

This could ideally be eventually solved by edit searches, but we are not really near to having collection filters for edit searches.

At least for now, having an extra "hide own edits" condition for collections seems like by far the easiest choice, and combines well with the status condition we already have in find_by_collection.

Testing

Manually, with my own private collection I happen to have a lot of edits in. You can always make some edits locally to test though :)

Collection edit lists cannot be filtered, and it can be very hard
to find others' edits to review on a collection where you are fairly active
yourself as an editor.

This could ideally be eventually solved by edit searches, but we are
not really near to having collection filters for edit searches.

At least for now, having an extra "hide own edits" condition for collections
seems like by far the easiest choice, and combines well with the status
condition we already have in find_by_collection.
@reosarevok reosarevok added the New feature Non urgent new stuff label Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New feature Non urgent new stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant