-
Notifications
You must be signed in to change notification settings - Fork 190
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
feat: add Unified REST API docs #3919
Conversation
👋 🤖 🤔 Hello! Did you make your changes in all the right places? These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.5/.
You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines. |
Added @pepopowitz for a DX review since this incorporates updates to the API reference docs via integrated explorer. |
@tmetzke 👀 did this go live... today? |
fe23a59
to
0c2c4ca
Compare
No, this was merged after the 8.6.0-alpha2 code freeze, so it will officially come with alpha3. |
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.
Per an internal conversation, @aleksander-dytko proposes we launch this as "Camunda 8 REST API" instead of "Unified REST API".
@tmetzke can you update this PR? And any supporting assets to reflect this name?
06633ed
to
22a8d9a
Compare
Adjusted the PR and the (internal) endpoint definition document. |
@tmetzke This looks good text-wise.
|
I wasn't aware that we have such a thing, I just looked at the existing tutorial now. |
docs/apis-tools/camunda-api-rest/specifications/activate-jobs.api.mdx
Outdated
Show resolved
Hide resolved
docs/apis-tools/camunda-api-rest/specifications/camunda-8-rest-api.info.mdx
Outdated
Show resolved
Hide resolved
* Adds the unified REST API explorer to the list of APIs.
Co-authored-by: Steven Hicks <[email protected]>
871bea2
to
771cdc1
Compare
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.
🌟 Bonus points for assisting with the other cleanup 🏆
@akeller, ready for merge from my end (couldn't remember if I hit squash and merge or if your team does it 😄) |
@tmetzke We are in this weird limbo period ahead of the release. You can merge this but risk releasing it if we need to do any releases before next week. If this is not a risk, you can merge. |
My most beloved phase of each month. Well, it would be a sneak peak at worst and the artifacts are also already built and released, so 🤷 You cannot access it in SaaS yet but it's also on Next. I'd assume we can live with this. |
YOLO let's go |
@tmetzke I'll add an issue to my backlog to create a tutorial for this, and I'll get this added to Postman next Tuesday 😄 |
Description
closes #3918
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).