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

Linking existing organisation with ROR id fails when ROR organisation was referenced elsewhere #457

Closed
cmeessen opened this issue Jul 27, 2022 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@cmeessen
Copy link
Contributor

cmeessen commented Jul 27, 2022

Steps to reproduce

  1. Create an organisation without linking it to the ROR id
  2. Create or edit a project, navigate to "Organisations"
  3. Under "Add organisation", type the name of the organisation you just added (will list RSD and ROR entries)
  4. Select the ROR entry
  5. Navigate back to Organisations -> The one you created in the first step
  6. open Organisation settings
  7. click the refresh button next to the ROR id field
  8. in the modal, select the corresponding organisation that you added in step 4
  9. Error message will appear: Failed to save. : duplicate key value violates unique constraint "organisation_ror_id_key"

/edit: what is happening is that when referencing the organisation in step 4, it will create a new one. So you cannot link two organisations with the same ROR (which is correct). At this point it would be handy to have delete function as mentioned in #448

@cmeessen cmeessen added the bug Something isn't working label Jul 27, 2022
@bakhshir
Copy link

bakhshir commented Jul 27, 2022

Yes, this is related to #442

@ewan-escience
Copy link
Collaborator

This is in principle expected behaviour. With #537, we also expect that this won't happen much anymore, even more so since the admin interface doesn't allow you to create an organisation without linking it to software or a project first.

It could be a problem when an organisation in the RSD db didn't have a ROR-id for a long time, and later does get one. We should allow for "merging" the two entries in the admin interface, but since this will be such a rare event, this is more for the long term.

@ewan-escience
Copy link
Collaborator

Now that admins can delete organisations since #1286, we consider this to be not an issue anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants