-
Notifications
You must be signed in to change notification settings - Fork 278
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
chore(content/update): Refactor and Amend Weld Message #1669
Conversation
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.
Where can I find the installation guide hint?
❓ I couldn't find a relevant section to add the hint. Where do you think the |
UpdateAfter applying the code review suggestion the formatting of the text didn't feel right. Here are the Before / After Screenshots to ease up the review process: PS: The same change was applied to the Patch Level Update Section |
Let's glue a note box to the installation guide where we repeat the information for Tomcat 10. Maybe something along the following lines:
Should be part of the 7.21 installation guide as well. |
Applied the suggested change. Will follow up with the |
- Tomcat Installation Guide - Migration Guide - Patch Level Section Co-authored-by: tasso94 <3015690+tasso94@users.noreply.github.com> Related-to: camunda/camunda-bpm-platform#4499
6f95ad8
to
93534cc
Compare
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.
Looks good now 🙂 👍 I replaced the word fix
with workaround
.
Content
Related-to: camunda/camunda-bpm-platform#4499