-
Notifications
You must be signed in to change notification settings - Fork 198
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
draft bpmn copilot #4735
draft bpmn copilot #4735
Conversation
👋 🤖 🤔 Hello, @christinaausley! 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. |
Thanks @christinaausley ! I'll take a look at this later this week 😄 |
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.
@ev-codes I added some thoughts. Please review them for accuracy and integrate them in. I do not need to re-review this doc
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.md
Outdated
Show resolved
Hide resolved
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.md
Outdated
Show resolved
Hide resolved
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.md
Outdated
Show resolved
Hide resolved
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.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.
Thanks @christinaausley ! I left a few points.
We should also revisit the screenshot in January closer to launch, as the UI and text may change slightly.
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.md
Outdated
Show resolved
Hide resolved
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.md
Outdated
Show resolved
Hide resolved
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.md
Outdated
Show resolved
Hide resolved
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.md
Outdated
Show resolved
Hide resolved
Thanks all for your feedback -- I will clean up this PR next week, with plans for a final review/any minor updates closer to January. |
This has been tidied based on the comments. I will revisit this PR in January to confirm the UI references and screenshots. |
@lmbateman @ev-codes Can you please have another look over these changes and confirm the UI? Would you like this to go out with alpha-3 next week? |
@christinaausley thanks for checking in! We won't be releasing this publicly for a few weeks. I'll do a final UI check closer to release. Also: @YanaSegal is now the primary designer working on this project. I'll add her as a reviewer closer to release. Thanks 😄 |
@YanaSegal can you confirm the capabilities and UI references outlined in this PR? Will this still be released this Thursday 1/23? |
@christinaausley we will only do an internal release to Camundi during the Kick-off Week. For internal releases, would you recommend releasing the documentation publicly already? For a release to external customers, we're aiming for |
@ev-codes I would not release the documentation publicly until the product is public as well. We can always pull the existing documentation into Confluence if you have an existing page or issue there already, if you would like. |
@YanaSegal can you confirm the capabilities and UI references outlined in this PR for release next week? |
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.
@christinaausley thanks! Docs look accurate overall.
Two small things:
- I pushed an updated screenshot
- In the UI, we use the capitalization BPMN/Camunda Copilot. Should we also use this in the technical docs? Whatever you think makes sense 😄
@YanaSegal is on FTO this week, but we can incorporate any changes she proposes when she's back Monday.
@ev-codes @mesellings Do we want to standardize on "copilot" lowercased? |
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.md
Outdated
Show resolved
Hide resolved
Hmm, I would have said if it's the product name, it would be BPMN Copilot, like Camunda Copilot, unless we're just informally referring to it as a 'copilot' (i.e. concept rather than a specific product)? Feels like Connectors/connectors again 😄
|
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.
@mesellings in the product and the upcoming release blog, this feature is labelled as BPMN Copilot.
There are plans in the 1-3 month horizon to offer a FEEL Copilot. In the future, we might want to release additional Copilots. That said, I haven't seen them referred to in the lower case, even when discussed internally.
As you note, Copilot seems to be the most common capitalization form in the industry.
I now have a preference to use the capitalized form "Copilot".
@HanselIdes what do you think?
docs/components/early-access/alpha/bpmn-copilot/bpmn-copilot.md
Outdated
Show resolved
Hide resolved
"Copilot" LGTM - I care that things are at least internally consistent |
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.
@christinaausley awesome work, thanks for all your work and patience!
This is ready for final review and merge 👍 |
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.
Thanks for catching the sidebar!
The preview environment relating to the commit b167f07 has successfully been deployed. You can access it at https://preview.docs.camunda.cloud/pr-4735/index.html |
Description
Relates to feedback iteration of the Web Modeler BPMN copilot with https://github.com/camunda/product-hub/issues/2511.
Before merging, must confirm:
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).