MIT-LCP / physionet-build

The new PhysioNet platform.
https://physionet.org/
BSD 3-Clause "New" or "Revised" License
56 stars 20 forks source link

Allow user accounts to be merged #857

Open tompollard opened 4 years ago

tompollard commented 4 years ago

It would be good to have a method for merging user accounts. For example:

elfeto commented 4 years ago

Right now it I merge abc@test2.com to abc@test.com, what I have is:

That is what I have done as of now. I am working right now on:

If the cloud information for user abc@test2.com exists I set it to None The profile information is kept the same. The user profile is kept.

I have two questions:

  1. How should I handle the CharField email in the user object? or should I just delete the user/profile objects?
    • I thought of prepending and appending "merged".
    • I was suggested to replace the @ for _ and append "@physionet.org"
  2. Is there anything else I might be missing right now?

PS. I cannot import anything from the project model in the user model.

bemoody commented 4 years ago

I'm not at all convinced that adding this feature would be wise or that it is a good solution to the problem.

The way Felipe has (partially) implemented it is a web interface for an administrator to manually select two accounts and merge them.

How do you intend to use this feature? How would you verify that the two accounts actually belong to the same person?