-
Notifications
You must be signed in to change notification settings - Fork 182
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
experimental: RPA documentation #4440
base: experimental-structure-PR
Are you sure you want to change the base?
experimental: RPA documentation #4440
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.6/.
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. |
@marstamm Is this meant to go into the |
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.
Feedback on the addition (some meta-feedback inside) ⏬
Add relevant links (i.e. link to the components from the description for easier access):
Do not make the "Robottic Process Automation" navigation entry a dead link. Instead, as I click on it I end up on the page describing it. Remove on of the sub-pages. Consider to call the page "RPA integration" (it should be clear that this is with Camunda).
See Desktop Modeler and some other pages that already have a proper navigation structure (where clicking a section leads to actual content, not just opens it in the navigation):
This does not seem to be properly formatted (the numbered list is part of the second bullet point):
Borders / padding around screenshots look odd, remove them:
Consider to link back to the "landing page" for RPA, to provide further context from the RPA framework library page:
Does not make sense to me:
I propose we make {variableName}
verbatime (code) syntax here, as it references a technical idenfifier (in the above script):
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.
Added a few comments with small improvements, nothing major from my side :)
docs/components/experimental/rpa/camunda-rpa-framework-library.md
Outdated
Show resolved
Hide resolved
docs/components/experimental/rpa/rpa-integration-with-camunda.md
Outdated
Show resolved
Hide resolved
docs/components/experimental/rpa/camunda-rpa-framework-library.md
Outdated
Show resolved
Hide resolved
docs/components/experimental/rpa/camunda-rpa-framework-library.md
Outdated
Show resolved
Hide resolved
docs/components/experimental/rpa/camunda-rpa-framework-library.md
Outdated
Show resolved
Hide resolved
docs/components/experimental/rpa/camunda-rpa-framework-library.md
Outdated
Show resolved
Hide resolved
I have resolved all lingering comments @marstamm -- let me know how this looks to you 👍 |
The preview environment relating to the commit c86c0a1 has successfully been deployed. You can access it at https://preview.docs.camunda.cloud/pr-4440/index.html |
Description
Relates to https://github.com/camunda/product-hub/issues/2412.
Incorporates RPA documentation into experimental features.
When should this change go live?
hold
label or convert to draft PR) CConPR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).