-
Notifications
You must be signed in to change notification settings - Fork 197
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
add overviews to bpmn elements #3137
Conversation
Related to https://github.com/bpmn-io/internal-docs/issues/805. @philippfromme Let me know what you think here. Once approved, I can backport as I see the issue is now closed. |
Need any help here? This looks stalled. |
@philippfromme @CatalinaMoisuc mind having a quick review here sometime this week? |
@christinaausley I will have a look today! 👀 |
👋 🤖 ✅ Looks like the changes were ported across versions, nice job! 🎉 You can read more about the versioning within our docs in our documentation guidelines. |
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.
Looking good now! 👏🏻
@philippfromme @christinaausley I assume that this PR can be merged since it was already reviewed by one bpmn.io core expect, no need for an extra review from my side, correct? |
Description
Related to https://github.com/bpmn-io/internal-docs/issues/805.
@philippfromme Let me know what you think here. Once approved, I can backport.
When should this change go live?
hold
label or convert to draft PR)?PR Checklist
/versioned_docs
directory, or they are not for an already released version./docs
directory (aka/next/
), or they are not for future versions.