-
Notifications
You must be signed in to change notification settings - Fork 195
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
docs(web-modeler): announce milestone to version renaming #4901
docs(web-modeler): announce milestone to version renaming #4901
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@wollefitz, is this labeled because it's already available in SaaS but will be in SM in alpha4? 🤔 maybe we need to discuss how to label things better for Web Modeler. |
Yes, exactly. In SaaS, we just continuously release the features of which most of them eventually will end up in SM.
I think so, too. Until now, we always applied SaaS changes to |
@camunda/tech-writers Just wanted to check in again on the status of this PR - do you think we can get it merged soon so that SaaS-users are already made aware of the naming changes within Web Modeler? |
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.
Made some formatting adjustments and added a few links here. One non-blocking comment. If you feel this would be helpful for users, I can certainly open an issue for it. If you don't feel users will struggle with this change, we can leave it as is for now.
I think it would be helpful to mention the former naming somewhere so that when using the search, you land on the new versions page 👍 Do you already have something in mind/would this be a small change or should we do it as a follow-up? |
@wollefitz I added a note in two places in my latest commit -- WDYT? |
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 the adjustment, sounds good! Let's get this merged 🙂
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 very minor (non-blocking) wording change for simplicity/translation, but good to go without the change 👍 🚀
Co-authored-by: Mark Sellings <mark.sellings@camunda.com>
@mesellings I applied your suggestion but now need another approving review 🫠 After that I'll 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.
Lgtm! 🚀
Description
Part of https://github.com/camunda/web-modeler/issues/12551 and follow-up for #4868
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).