doc:web: update openapi.yaml for 1.50#2554
Conversation
|
Thanks for the update! I would prefer it without the CHANGES.md changes. I update those as part of the release process. |
|
Now is it right? |
|
Not quite; you have added a commit deleting CHANGES.md. You should have reverted the CHANGES.md part of the original commit and amended that commit with that reversal (it's hard to describe since there are lots of git tools). End result: a single commit just changing openapi.yaml. Perhaps easier: abandon both commits and redo the openapi one, then force-push to this PR branch. |
97f41d9 to
2fd9715
Compare
|
Thank you for your patience; I am not used to doing this kind of stuff. |
|
One minor doubt, if you do not care: was this the correct way to update the docs, or should I have done it on the original open issue? |
|
Looks good - thank you! A pull request like this was the best way. |
Update the OpenAPI specs because I wanted to try the web module and saw #2459.