-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
Upgrade to JHipster 7 #185
Conversation
avdev4j
commented
Apr 12, 2021
•
edited
Loading
edited
- Upgrade to JHipster 7
- remove UAA
- remove setupXXVersion
- Add new lifecycle methods
- update Vue health check [v7] Update Vue health checks screens #98
- Use composing function in the app generator instead of defining the composed subgen in the configuring() method https://github.com/jhipster/generator-jhipster/blob/main/generators/app/index.js#L387
- Use composing function in the entity generator instead of defining the composed subgen in the configuring() method
- Fix the todo here https://github.com/jhipster/generator-jhipster-quarkus/blob/main/generators/entity-server/index.js#L68
- The CI is green
3da592b
to
4ff9e37
Compare
SonarCloud Quality Gate failed. 0 Bugs No Coverage information |
SonarCloud Quality Gate failed. 0 Bugs No Coverage information |
Anything I can do to help with this? I'd like to create a screencast of this post. However, I think it'd be nice to update it for Quarkus 2.0 first. Are there plans to release a new version of this blueprint with Quarkus 2.0 support in the next week or two? |
Hi Matt, The entity subgen is quite complexe to migrate due to several changes in JHipster. If you want you can contribute on this part. There is also the vue part to modify but it's a minor change. We plan to release a new version (a beta has been released with Quarkus 2 and SQL DB) including JHipster and the whole support of Quarkus 2 (redis, mongo...). Not sure it will be done in one or two weeks. |
I added a bug bounty that might help. @mshima was a big help in migrating the Ionic module to JHipster 7. jhipster/generator-jhipster-ionic#482 |
As far as I know the main problem could be resolved by checking how the context is loaded inside the BP and ensure the composing part is made as expected. |
@avdev4j Can you fix the conflicts please? |
…hat quarkus master works against generator-jhipster master
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
fixed @mraible |
Any chance you can help with this upgrade, @mshima? If not, I can take a look next week. |
I will take a look. |
I don't have permission to push to this branch, created #223. |
@avdev4j can we split the bounty in half? $250 each? |
I'm sorry @mshima I totally forgot to answer you, yes let's go for it! 250 each |
@pascalgrimaud half bounty claimed https://opencollective.com/generator-jhipster/expenses/59119. |
@mshima : approved |
Thanks @pascalgrimaud |