-
Notifications
You must be signed in to change notification settings - Fork 205
feat(desktop-modeler): create template versioning section #5778
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
base: main
Are you sure you want to change the base?
Conversation
👋 🤖 🤔 Hello, @nikku! 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.7/.
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. |
…ning-templates.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
docs/components/modeler/desktop-modeler/element-templates/defining-templates.md
Outdated
Show resolved
Hide resolved
…ning-templates.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…ning-templates.md Co-authored-by: Simon <[email protected]>
docs/components/modeler/desktop-modeler/element-templates/defining-templates.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/desktop-modeler/element-templates/defining-templates.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/desktop-modeler/element-templates/defining-templates.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/desktop-modeler/element-templates/defining-templates.md
Outdated
Show resolved
Hide resolved
docs/components/modeler/desktop-modeler/element-templates/defining-templates.md
Outdated
Show resolved
Hide resolved
I like the general direction but following the link and seeing the graphics at https://github.com/bpmn-io/element-templates/blob/main/docs/LIFE_CYCLE.md#overview I think we could improve that (I find it hard to comprehend without screenshots) or just link to the existing documentation that covers the entire lifecycle (though engine compatibility is currently missing). I think we need to cover 5 different states: https://miro.com/app/board/uXjVI1a7lHU=/ |
…ning-templates.md Co-authored-by: Christina Ausley <[email protected]>
…ning-templates.md Co-authored-by: Christina Ausley <[email protected]>
…ning-templates.md Co-authored-by: Christina Ausley <[email protected]>
…ning-templates.md Co-authored-by: Christina Ausley <[email protected]>
…ning-templates.md Co-authored-by: Christina Ausley <[email protected]>
@philippfromme What is hard to comprehend about the graphics? Keep in mind this is developer, not user documentation. |
I guess it's just me. 😅 |
@philippfromme I will keep an eye on this page with PushFeedback (which tracks incoming feedback from users on each page) to see if we get any additional thoughts on this. For example: ![]() On the backend I can see if anyone leaves comments with their thumbs up/down as well. @nikku Is this still a WIP or ready to be moved out of draft and merged? |
Description
This PR ensures improves the existing template definition documentation:
engine
andversion
)When should this change go live?
bug
orsupport
label)available & undocumented
label)hold
label)low prio
label)PR Checklist
/docs
directory (version 8.8)./versioned_docs
directory.@camunda/tech-writers
unless working with an embedded writer.