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

Remove child work search filter #542

Open
grahamjevon opened this issue May 7, 2024 · 1 comment
Open

Remove child work search filter #542

grahamjevon opened this issue May 7, 2024 · 1 comment

Comments

@grahamjevon
Copy link
Collaborator

grahamjevon commented May 7, 2024

Noticed during testing for #451

Problem

When creating relationships between works, four filters are applied to the lookup search filter. Three of these filters make sense and should be kept. These are: :show_only_other_works, :show_only_works_not_child, :show_only_works_not_parent

The following filter, however, is problematic :show_only_resource_depositord_by_current_user. This filter means that if I upload a work, Nora would not be able to add it as the child of another work using? So if I uploaded a work today and leave the organisation tomorrow, thereafter no-one will be able to add that work as the child of another work?

This problematic filter might be appropriate if we had a self-deposit system (e.g. every member of staff had login credentials and uploaded their own outputs as works). However, it is problematic when we only have a limited number of administrators doing all the uploading.

Solution

Keep the first three filters and remove the fourth filter: :show_only_resource_depositord_by_current_user.

@grahamjevon
Copy link
Collaborator Author

Staging works as expected. I can create relationships between works regardless of whether I deposited the work. For example, I could add a work added by Jenny to another work added by Jenny.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

1 participant