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

#check_admin_set is overly cautious #76

Open
aploshay opened this issue Sep 8, 2021 · 0 comments
Open

#check_admin_set is overly cautious #76

aploshay opened this issue Sep 8, 2021 · 0 comments

Comments

@aploshay
Copy link
Contributor

aploshay commented Sep 8, 2021

Profile deletion attempts first call check_admin_set, which returns true if the Profile is affiliated to any admin sets that contain any works. This results in a condition where a newly-created, but entirely-unused profile, refuses deletion claiming that it is in use, even if no works have been created with the profile or updated to use it.

Suggest softening the condition as done in IU-Libraries-Joint-Development/essi@0929763

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

No branches or pull requests

1 participant