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

Upgrade to JHipster 7 #185

Closed
wants to merge 20 commits into from
Closed

Upgrade to JHipster 7 #185

wants to merge 20 commits into from

Conversation

avdev4j
Copy link
Contributor

@avdev4j avdev4j commented Apr 12, 2021

@avdev4j avdev4j force-pushed the 7.x-features branch 2 times, most recently from 3da592b to 4ff9e37 Compare April 12, 2021 10:18
@sonarcloud
Copy link

sonarcloud bot commented Apr 12, 2021

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
21.5% 21.5% Duplication

@sonarcloud
Copy link

sonarcloud bot commented Jun 16, 2021

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
10.9% 10.9% Duplication

@mraible
Copy link
Contributor

mraible commented Aug 4, 2021

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?

@avdev4j
Copy link
Contributor Author

avdev4j commented Aug 4, 2021

Hi Matt,
I'm working on it but I'm struggling a bit with the entity subgen. I think every other things are working.

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.

@mraible mraible added $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ $500 https://www.jhipster.tech/bug-bounties/ labels Aug 4, 2021
@mraible
Copy link
Contributor

mraible commented Aug 4, 2021

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

@avdev4j
Copy link
Contributor Author

avdev4j commented Aug 4, 2021

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.

@mraible
Copy link
Contributor

mraible commented Aug 10, 2021

@avdev4j Can you fix the conflicts please?

@sonarcloud
Copy link

sonarcloud bot commented Aug 13, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
1.2% 1.2% Duplication

@avdev4j
Copy link
Contributor Author

avdev4j commented Aug 13, 2021

fixed @mraible

@mraible
Copy link
Contributor

mraible commented Aug 13, 2021

Any chance you can help with this upgrade, @mshima? If not, I can take a look next week.

@mshima
Copy link
Member

mshima commented Aug 13, 2021

I will take a look.

@mshima
Copy link
Member

mshima commented Aug 15, 2021

I don't have permission to push to this branch, created #223.

@mshima mshima mentioned this pull request Aug 18, 2021
3 tasks
@mraible mraible closed this in #223 Aug 20, 2021
@mshima
Copy link
Member

mshima commented Oct 5, 2021

@avdev4j can we split the bounty in half? $250 each?

@avdev4j
Copy link
Contributor Author

avdev4j commented Nov 15, 2021

I'm sorry @mshima I totally forgot to answer you, yes let's go for it! 250 each

@avdev4j
Copy link
Contributor Author

avdev4j commented Nov 15, 2021

@pascalgrimaud
Copy link
Member

@avdev4j : approved
Waiting yours, @mshima

@mshima
Copy link
Member

mshima commented Dec 22, 2021

@pascalgrimaud
Copy link
Member

@mshima : approved

@mshima
Copy link
Member

mshima commented Dec 22, 2021

Thanks @pascalgrimaud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
$$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ $500 https://www.jhipster.tech/bug-bounties/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants