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

Handle merging people #23

Open
fpagnoux opened this issue May 17, 2022 · 2 comments
Open

Handle merging people #23

fpagnoux opened this issue May 17, 2022 · 2 comments

Comments

@fpagnoux
Copy link
Member

fpagnoux commented May 17, 2022

@mprat I think this feature has been broken for a while on ui-v1, so I couldn't test it to figure out exactly what it does.
Is the idea that we let the logged in desk worker pick which affiliation / name / primary email they want to use for the new merged person profile?

@mprat
Copy link
Member

mprat commented May 17, 2022

I was just about to comment here - I had started fixing this while I was doing the implementation of the auth API for DavidCain back in September (see my WIP CR here: https://github.com/mitoc/gear-db-django/pull/347) but it exposed some issues, which is why I shifted to the model cleanup.

I'd recommend holding off on this for a little bit until the other WIP CRs land and we can focus on this one. The model cleanup CR needs to be merged first since the fixes there will enable a robust solution here.

Edited: blocked by https://github.com/mitoc/gear-db-django/issues/240

@fpagnoux
Copy link
Member Author

That works for me!

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

2 participants