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

Transfer fails to create new admin contact in case of shared contacts #1808

Closed
vohmar opened this issue Jan 21, 2021 · 0 comments · Fixed by #1810
Closed

Transfer fails to create new admin contact in case of shared contacts #1808

vohmar opened this issue Jan 21, 2021 · 0 comments · Fixed by #1810
Labels
bug Tests prepared TDD: autotests prepared

Comments

@vohmar
Copy link
Contributor

vohmar commented Jan 21, 2021

In case of domain transfer to another registrar new contact objects are created for the new registrar to give new registrar instant access to registrant data and possibly reduce the need to create new contacts. .ee registry system reuses existing objects in transfer whenever possible.

  1. Registry system checks if there is an existing contact object with identical data already present for the new registrar and uses that if one is found
  2. If there is no such object new object is automatically created
  3. in case objects have been reused by the previous registrar the freshly created object should now be reused for other roles as well

third point is where the process fails.

  • In case admin and tech are shared - replacing the object for the admin contact fails
  • in case registrant, admin and tech are shared - replacing tech object fails (same object is successfully used for registrant and admin)
  • everything works as expected in case registrant and admin is shared
  • everything work as expected in case registrant and tech is shared

so to simplify the issue is there in case admin and tech objects are shared probably because in this case the relation is described in the same domain_contact table.

@vohmar vohmar added the bug label Jan 21, 2021
@OlegPhenomenon OlegPhenomenon self-assigned this Jan 21, 2021
@OlegPhenomenon OlegPhenomenon added the Tests prepared TDD: autotests prepared label Jan 21, 2021
@OlegPhenomenon OlegPhenomenon removed their assignment Jan 21, 2021
@yulgolem yulgolem self-assigned this Jan 22, 2021
@yulgolem yulgolem assigned OlegPhenomenon and unassigned yulgolem Jan 22, 2021
@OlegPhenomenon OlegPhenomenon removed their assignment Jan 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Tests prepared TDD: autotests prepared
Projects
None yet
3 participants