Skip to content

MC-15611: doc for create and update dns record#421

Merged
AnKhangC merged 7 commits intocmc-devfrom
feature/MC-15611-add-dns-record
Aug 17, 2021
Merged

MC-15611: doc for create and update dns record#421
AnKhangC merged 7 commits intocmc-devfrom
feature/MC-15611-add-dns-record

Conversation

@AnKhangC
Copy link
Copy Markdown

@AnKhangC AnKhangC commented Aug 11, 2021

Fixes MC-15611

Fixes MC-15612

Changes made

-Doc for create dns record
-Doc for update dns record

Related PRs

#### Update a DNS record

```shell
curl -X POST \
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this is a put

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Co-authored-by: Alain Lavoie <56133634+fyrnaga@users.noreply.github.com>
Copy link
Copy Markdown

@fyrnaga fyrnaga left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown

@HassanHaidar HassanHaidar left a comment

Choose a reason for hiding this comment

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

Looks good 👍


Query Params | &nbsp;
---- | -----------
`zoneId`<br/>*UUID* | The ID of the zone for which we want to delete the record. This parameter is required.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

delete -> create

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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


Query Params | &nbsp;
---- | -----------
`zoneId`<br/>*UUID* | The ID of the zone for which we want to delete the record. This parameter is required.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

update the record

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@AnKhangC AnKhangC merged commit 0523e41 into cmc-dev Aug 17, 2021
@AnKhangC AnKhangC deleted the feature/MC-15611-add-dns-record branch August 17, 2021 18:42
jenkins-cloudops pushed a commit that referenced this pull request Aug 17, 2021
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

Successfully merging this pull request may close these issues.

4 participants