Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Included docs for POST,PUT,DELETE for v3,v4,v5 of statuses and statusesID#7588

Merged
rimashah25 merged 11 commits intoapache:masterfrom
heneriknguyen:status_docs
Jun 29, 2023
Merged

Included docs for POST,PUT,DELETE for v3,v4,v5 of statuses and statusesID#7588
rimashah25 merged 11 commits intoapache:masterfrom
heneriknguyen:status_docs

Conversation

@heneriknguyen
Copy link
Copy Markdown
Contributor

Closes: #6318


Which Traffic Control components are affected by this PR?

  • Documentation

What is the best way to verify this PR?

Ensure the docs are working correctly.

If this is a bugfix, which Traffic Control versions contained the bug?

7.0.1

PR submission checklist

@ocket8888 ocket8888 added bug something isn't working as intended documentation related to documentation low impact affects only a small portion of a CDN, and cannot itself break one labels Jun 22, 2023
@rimashah25 rimashah25 requested a review from ocket8888 June 26, 2023 16:33
Comment thread docs/source/api/v4/statuses_id.rst
Comment thread docs/source/api/v5/statuses_id.rst
@rimashah25 rimashah25 requested a review from ocket8888 June 27, 2023 16:43
Copy link
Copy Markdown
Contributor

@jagan-parthiban jagan-parthiban left a comment

Choose a reason for hiding this comment

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

In this PR, for V4 and V5, this is the docs structure added by you,

statuses
- GET
- POST
- PUT
- DELETE

statuses/{{ID}}
- GET
- PUT
- DELETE

I think we have to remove PUT and DELETE from statuses and remove GET from statuses/{{ID}}. It should be like any other documentation,

statuses
- GET
- POST

statuses/{{ID}}
- PUT
- DELETE

References:
divisions
divisions/{{ID}}

Comment thread docs/source/api/v5/statuses.rst Outdated
Copy link
Copy Markdown
Contributor

@rimashah25 rimashah25 left a comment

Choose a reason for hiding this comment

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

LGTM

@rimashah25 rimashah25 merged commit 5d2d866 into apache:master Jun 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug something isn't working as intended documentation related to documentation low impact affects only a small portion of a CDN, and cannot itself break one

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing docs for DELETE method of /statuses/{{ID}}

4 participants