Actions
Task #2000
closedUpgrade dependency on core to version 10.0.0
Start date:
12/18/2019
Due date:
% Done:
100%
Estimated time:
Owner:
Actions
Added by Peter Štrunc about 5 years ago. Updated almost 5 years ago.
100%
I upgraded BE and FE dependencies and tested the functionality. I also fixed few sonar bugs and refactored some unnecessary imports. Code is in a separate branch on master 2.0.0-rc. I will find someone to do the review after the holidays.
Hi @tomiskar, would you please review this task? Thanks
I did code review for migration and install module into CzechIdM 10, module is available, awesome, thx!
Review notes:
- PasswordResetSwaggerConfig#exampleApi - duplicate bean name with example module - module cannot start - rename method to #pwdresetApi.
- product dependency should have provided scope
- module should have snapshot version in branch
- warnings is present in module (Assert.notNull without message etc.)
- backend sources should be moved into Realization/backend/idm-pwdreset folder (see archetype conventions https://github.com/bcvsolutions/idm-module-archetype#czechidm-module-archetype)
- html documetation is missing (see extras module pom as example)
- frontend dependencies (react-google-recaptcha, react-async-script) are already defined in product (app/package.json), could be removed from this module (=> reuse library from product).
I fixed all mentioned issues. I had to do some magic with git branches.
There is 10.0.0 core version in both master and develop branches. If there is a need to finish unfinished version 1.7.0-snapshot, then there is a separate branch for that. Otherwise we will continue in develop with version 2.1.0.
Version is released. CLosing the ticket.