Skip to content

OpenAPI spec update from glideapps/glide#29066 #11

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

Merged
merged 10 commits into from
Aug 28, 2024
Merged

Conversation

chkn
Copy link
Contributor

@chkn chkn commented Aug 21, 2024

Includes the OpenAPI spec updates from glideapps/glide#29066.

Please do the following:

  1. Review the spec changes in this PR and ensure they are documented appropriately. If needed, push any additional changes to glideapps/glide#29066.
  2. If the above should be accompanied by changes to other files in this repo, push them to this branch.
  3. When ready, enable auto-merge. Then, when glideapps/glide#29066 is released to prod, the blocked:unreleased label will be removed, and this PR will be automatically merged and published as well.

created with ❤️ by the API docs workflow

@chkn chkn added the blocked:unreleased The changes documented in this PR have not yet been released label Aug 21, 2024
Copy link
Contributor

This PR is being prevented from merging because it has one or more of these labels applied: blocked:unreleased.

Copy link
Contributor

@rwdaigle rwdaigle left a comment

Choose a reason for hiding this comment

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

Awesome! Can we also revert this commit, which was intended to be temporary, as part of this update?

6d10067

Also, let's update the changelog with an entry for the stash UUID requirement and the change from POST to PUT?

And... I wonder if we should document the new error state around invalid stash id/serials? Maybe on our error page?

@chkn chkn force-pushed the api-update-glide-pr-29066 branch from aa8e6e7 to 38d3473 Compare August 22, 2024 00:27
@chkn chkn changed the base branch from main to add-devcontainer August 22, 2024 00:27
@chkn chkn force-pushed the api-update-glide-pr-29066 branch from 38d3473 to 64c32a9 Compare August 22, 2024 00:30
@chkn chkn force-pushed the api-update-glide-pr-29066 branch from bb618e9 to c12bab2 Compare August 22, 2024 13:48
@chkn chkn changed the base branch from add-devcontainer to main August 22, 2024 13:48
@chkn chkn requested a review from rwdaigle August 22, 2024 16:34
@chkn chkn enabled auto-merge August 22, 2024 18:19
Copy link
Contributor

It looks like you've made changes to the API docs! 🙌
☑️ The changelog has been updated 🎉
👉 You may want to preview the changes in a Codespace

@chkn chkn removed the blocked:unreleased The changes documented in this PR have not yet been released label Aug 28, 2024
@chkn chkn merged commit 0d9e9f8 into main Aug 28, 2024
4 of 5 checks passed
@chkn chkn deleted the api-update-glide-pr-29066 branch August 28, 2024 16:04
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.

2 participants