-
Notifications
You must be signed in to change notification settings - Fork 140
3 - Add API docs for stack deployments and stack deployment groups #910
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
34779bb
to
b3a9448
Compare
Howdy @mrinalirao! Heads up that we will be migrating the content from So a heads up, after the migration happens I'll close this PR and will let you know when you are good to recreate it! Apologies about the extra work you have to do though! |
9b150d3
to
b5a6432
Compare
b3a9448
to
f6046d1
Compare
b5a6432
to
36b8d94
Compare
Hello @mrinalirao! The UDR migration has occurred, and the HCP Terraform documentation version now lives in the unified documentation repository. I'm going to close this PR since we are no longer serving this content from this repo, but please feel free to recreate your PR in the UDR repository within the HCP Terraform documentation and tag me if you need a review: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know the PR is closed but the content is good.
|
||
### Sample Response | ||
|
||
/* TODO: For reviewer - check if stack-deployment-runs should belong in the links as you need the stack deployment group id to fetch it */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we leave this todo comment up? I think we're not going to render the runs here but I'm sure that's up in the air to some degree so we can double check when the apis are more finalized.
| Parameter | Description | | ||
|----------------|-----------------------------------------------------------------------------| | ||
| `page[number]` | **Optional.** If omitted, the endpoint will return the first page. | | ||
| `page[size]` | **Optional.** If omitted, the endpoint will return 20 policy sets per page. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
20 stacks
| Parameter | Description | | ||
| ------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | ||
| `page[number]` | **Optional.** If omitted, the endpoint will return the first page. | | ||
| `page[size]` | **Optional.** If omitted, the endpoint will return 20 policy sets per page. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
20 stacks
This is a copy of the PR created here: : hashicorp/terraform-docs-common#910 The comments have been addressed. NOTE: This PR will remain in draft as this cannot be merged before stacks goes GA. The API documentation has been divided into multiple PRs to simplify the review process. As each of these PRs is approved, it will be merged into this branch mr/stacks-api-ga
What
Note: The following Stacks PRs need to be approved first and in the following order:
This PR adds API docs for basic stacks configuration APIs.
https://hashicorp.atlassian.net/browse/TF-26279
NOTE: This PR will remain in draft as this cannot be merged before stacks goes GA.
The API documentation has been divided into multiple PRs to simplify the review process. As each of these PRs is approved, it will be merged into mr/stacks-api-ga branch.
API's introduced in this PR:
Why
In preparation for stacks going GA, we need to document all the public facing API's for stacks
Merge Checklist
If items do not apply to your changes, add (N/A) and mark them as complete.
Pull Request
Content
website/redirects.js
for moved, renamed, or deleted pages.Reviews