Skip to content

Add support for registrant APIs#159

Merged
8 commits merged into
mainfrom
feature/create-registrant-change
Aug 31, 2023
Merged

Add support for registrant APIs#159
8 commits merged into
mainfrom
feature/create-registrant-change

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 22, 2023

This PR introduces 5 new endpoints that are part of the domain contact management API that was introduced in dnsimple/dnsimple-developer#501.

The following endpoints have been added:

  • listRegistrantChanges: GET /{account}/registrar/registrant_changes
  • createRegistrantChange: POST /{account}/registrar/registrant_changes
  • checkRegistrantChange: POST /{account}/registrar/registrant_changes/check
  • getRegistrantChange: GET /{account}/registrar/registrant_changes/{registrantchange}
  • deleteRegistrantChange: DELETE /{account}/registrar/registrant_changes/{registrantchange}

Belongs to https://github.com/dnsimple/dnsimple-business/issues/1729

@ghost ghost added the enhancement label Aug 22, 2023
@ghost ghost self-requested a review August 22, 2023 05:08
@ghost ghost self-assigned this Aug 22, 2023
Copy link
Copy Markdown
Contributor

@DXTimer DXTimer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only the create registrant change endpoint has been added please add all 5 endpoints for the registrant change API: list, check, create, show, delete.

Tests are missing.

The code should be moved out of the Registrar.java file into its own file.

@ghost ghost changed the title Add support to create a registrant change Add support for registrant APIs Aug 23, 2023
@ghost ghost requested a review from DXTimer August 29, 2023 14:37
@DXTimer DXTimer requested a review from AGS4NO August 30, 2023 10:46
@ghost ghost merged commit 3c3727e into main Aug 31, 2023
@ghost ghost deleted the feature/create-registrant-change branch August 31, 2023 03:26
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant