Skip to content
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

fix(changelog): fix missing Update Tool changelog (2024.3) #2928

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

educhastenier
Copy link
Contributor

Also documents to new --update-indexes option

Also documents to new `--update-indexes` option
Copy link
Contributor

github-actions bot commented Dec 5, 2024

📝 Contribution Summary

To merge this Pull Request, you need to check your updates with the following URL.

🔗 Updated pages

Note

The following pages were updated, please ensure that the display is correct:

@bonita-ci
Copy link
Contributor

bonita-ci commented Dec 5, 2024

🎊 PR Preview 5d275dd has been successfully built and deployed to https://bonitasoft-bonita-doc-deploy-pr-2928.surge.sh

🕐 Build time: 0.009s

🤖 By surge-preview

== 3.6.1 - December 5th, 2024 - Bonita Update Tool changes
Bonita Update Tool handles updates from all Bonita 7.10 versions up to Bonita 2024.3 (10.2) version.

* [PERFORMANCE] This patch version fixes update versions 9.0 and all of the above, to fix database indexes, in order to improve performance.
Copy link
Contributor

@ugaston ugaston Dec 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would say something like:
[PERFORMANCE] This patch version fixes database indexes in order to improve performance, when updating to versions 9.0 and above.

Bonita Update Tool handles updates from all Bonita 7.10 versions up to Bonita 2024.3 (10.2) version.

* [PERFORMANCE] This patch version fixes update versions 9.0 and all of the above, to fix database indexes, in order to improve performance.
* [PERFORMANCE] it also introduces a new option `-i` or `--update-indexes` to fix the indexes of the database. This command can be launched without updating Bonita to a newer version. It can also be launched several times without risk: it does nothing if not necessary.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, in accordance with my proposition for the first point, I would have this one read:
[PERFORMANCE] it also introduces a new option -i or --update-indexes to fix the indexes of the database without actually updating Bonita to a newer version. It can also be launched several times without risk: it does nothing if not necessary.

Copy link
Contributor

@ugaston ugaston left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A bit of rephrasing suggested.
Else, it looks fine to me.

@educhastenier educhastenier requested a review from ugaston December 5, 2024 17:14
Copy link
Contributor

@ugaston ugaston left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome.
Thanks a lot, Manu!

@educhastenier educhastenier merged commit e558634 into 2024.3 Dec 5, 2024
5 checks passed
@educhastenier educhastenier deleted the chore/RN/fix_update_tool_changelog_2024.3 branch December 5, 2024 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants