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.

Sign up for GitHub

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

Jump to bottom

Update Delete User API Endpoint #6697

Closed
djbrooke opened this issue Feb 26, 2020 · 0 comments · Fixed by #6747
Closed

Update Delete User API Endpoint #6697

djbrooke opened this issue Feb 26, 2020 · 0 comments · Fixed by #6747
Assignees
@sekmiller

Comments

@djbrooke
Copy link
Contributor

djbrooke commented Feb 26, 2020 •
edited

#4475 proposes adding a delete button to the dashboard. We want this eventually, but there's some value that can be provided by updating our existing Delete User API to handle a few of the more simple cases. Specifically:

  • Test users
  • Users that haven't done anything that prevents their deletion in Dataverse

Over the past few months I've received deletion requests, and when I go to @scolapasta he has to run a few DB queries to determine if the user can be deleted or not. I'd like to be able to delete the user myself using an API, and, if the user cannot be easily deleted, I want to receive messaging that the user can't be deleted.
@djbrooke djbrooke moved this from Ready 🙋 to IQSS Sprint 2/26 - 3/11 in IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) Feb 26, 2020
@sekmiller sekmiller moved this from IQSS Sprint 2/26 - 3/11 to IQSS Team Dev 💻 in IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) Feb 27, 2020
@sekmiller sekmiller self-assigned this Feb 27, 2020
@djbrooke djbrooke moved this from IQSS Team - In Progress 💻 to IQSS Sprint 2/26 - 3/11 in IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) Mar 5, 2020
sekmiller added a commit that referenced this issue Mar 10, 2020
@sekmiller
#6697 delete user api update
567d4d4
@sekmiller sekmiller moved this from IQSS Sprint 2/26 - 3/11 to IQSS Team - In Progress 💻 in IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) Mar 11, 2020
@sekmiller sekmiller self-assigned this Mar 11, 2020
sekmiller added a commit that referenced this issue Mar 13, 2020
@sekmiller
#6697 add test for roles, refactor
83a47c9
sekmiller added a commit that referenced this issue Mar 13, 2020
@sekmiller
#6697 fix delete user endpoint
92a76e8
sekmiller added a commit that referenced this issue Mar 16, 2020
@sekmiller
#6697 more delete user tests bundle update
4d64f85
sekmiller added a commit that referenced this issue Mar 17, 2020
@sekmiller
#6697 add integration tests
ac67f2c
sekmiller added a commit that referenced this issue Mar 17, 2020
@sekmiller
#6697 fix failing test
9ecc40c
sekmiller added a commit that referenced this issue Mar 17, 2020
@sekmiller
#6697 add test for user has role assigned
89f5d8e
sekmiller added a commit that referenced this issue Mar 18, 2020
@sekmiller
#6697 refactor error messages
a909444
sekmiller added a commit that referenced this issue Mar 18, 2020
@sekmiller
#6697 add comments
44df633
@sekmiller sekmiller mentioned this issue Mar 18, 2020
Merged
sekmiller added a commit that referenced this issue Mar 18, 2020
@sekmiller
#6697 add doc for delete user
73cce68
@sekmiller sekmiller moved this from IQSS Team - In Progress 💻 to Code Review 🦁 in IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) Mar 18, 2020
sekmiller added a commit that referenced this issue Mar 23, 2020
@sekmiller
#6697 delete items that can be deleted
b85ce4a
sekmiller added a commit that referenced this issue Mar 23, 2020
@sekmiller
#6697 update doc wording
ea4d886
sekmiller added a commit that referenced this issue Mar 25, 2020
@sekmiller
#6697 make world map token cascade delete
b000c78
sekmiller added a commit that referenced this issue Mar 25, 2020
@sekmiller
#6697 add notifications to cascade delete
c7695fb
sekmiller added a commit that referenced this issue Mar 25, 2020
@sekmiller
#6697 fix comment
8a1cc8a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sekmiller sekmiller

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

6697 delete user api IQSS/dataverse
2 participants
@sekmiller @djbrooke