-
Notifications
You must be signed in to change notification settings - Fork 274
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
7.22 add tomcat 10 documentation #1647
Conversation
Tomcat Manual Installation Section
Tomcat Manual Installation Section
The above sections need to include both camunda-webapp-tomcat and camunda-webapp-tomcat-jakarta artifacts Tomcat Manual Installation Section
Add missing section "Migrate Java webapp plugins"
# Conflicts: # content/update/minor/721-to-722/_index.md
Co-authored-by: Daniel Kelemen <[email protected]>
Tomcat 9 & Tomcat 10.1 Link
This change is to emphasize: - the benefits of tomcat 10 - to stress that this version will be based on tomcat 10 (distro,docker) code-review point
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! Added two more notes.
Co-authored-by: Daniel Kelemen <[email protected]>
UpdateReview is complete. Will proceed to merge this documentation PR after the main code PR is merged. |
@danielkelemen I've added the Patch level update section addition in the docs for informing users of Tomcat 10 module support for the 7.21 Backport. Once the above is finalised after the review, i can proceed to add the above section also to the |
Add the reference to the patch level update section
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 overall. But I have some review hints:
Co-authored-by: tasso94 <[email protected]>
Code Review point addressed
Related-to: camunda/camunda-bpm-platform#4390