-
Notifications
You must be signed in to change notification settings - Fork 184
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
Document Deploy and Play for process applications #4367
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. |
@HanselIdes I've created a draft of the edits I think might be required for this work - please can you check to see what else we might need to mention here? I also wasn't sure about whether to rename the page, what do you think? |
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
Thanks for the feedback @HanselIdes - I've amended the page (including the title) to hopefully cover the things you have requested - let me know if more clarification is needed? (NB: Give the preview environment 20 mins to update) |
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.
This reorganization makes a lot of sense. Thank you @mesellings !
…ithub.com/camunda/camunda-docs into mes-4182-play-deploy-process-applications
@HanselIdes Please re-approve as I made a few small edits (meta description) and updates to the main page 👍 |
🧹 Preview environment for this PR has been torn down. |
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.
LGTM - one minor grammatical comment about comma placement
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
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.
LGTM
@camunda/tech-writers Please review to merge this week for 8.6 👍 |
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
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.
A little feedback on adding context/incorporating the user journey here, with some added explanations.
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/web-modeler/deploy-process-application.md
Outdated
Show resolved
Hide resolved
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.
LGTM (still)
@christinaausley @conceptualshark Please re-review for approval - great feedback - I have incorporated it all pretty much in one way or another, thank you! 👍 🚀 |
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.
👍 LGTM! Nice improvements to this section.
* Draft edits to include Play * Review edits * Edits for title change * Fix link and comma * Fix comma * TW review edits
Description
Relates to and closes #4182.
As per review comments on PR OOTB PDLC Happy Path documentation, this PR update the "Deploy and Run process applications" page for 8.6.
Specifically:
Acceptance criteria
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).