-
Notifications
You must be signed in to change notification settings - Fork 499
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
Add an admin API to merge accounts #5514
Comments
Currently the command only does the merger of roleAssignments
I was hoping to get this done before my 2 weeks away from Dataverse, but it turned out to be a bit more complicated than I thought. The code still needs tests and all the objects connected to the objects deleted in the command need to be altered. I'm not sure which need their foreign keys changed and which just need to be deleted. On friday Gustavo pointed me to https://github.com/IQSS/dataverse/blob/develop/scripts/migration/scrub_duplicate_emails.sql as reference for how we did this in the past but I didn't get to dig in. Specifically this part:
It may be possible to manage this instead by using cascade inside java, investigating that was going to be my next step. |
Currently contains two superUser commands
I'll use this ticket to record both rename and merge issues.
[x] 2. Cannot log in to merged account. No error in logs but ui does not allow log in as if incorrect password. Was not able to reproduce. [x] 3. Cannot merge an account with an open file access request into another, throws error. [x] 4. Cannot merge an account with a group permission into another. [x] 5. Cannot merge an oauth account into a builtin, error. [x] 6. Cannot merge a shib user into a builtin, error. [x] 7. Not sure of the rules around guestbook and renaming/merging. When merge 2 accounts that have downloaded the same file (no guestbook) download response still shows old account info. |
We want to add an admin API that can be used to merge two acounts (i.e. move one account into another). This would require changing foreign keys as well as dealing with the role assignment table.
The text was updated successfully, but these errors were encountered: