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.

Sign up for GitHub

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

Jump to bottom

Feature/vue issue158 #211

Merged
24 commits merged into from
Mar 31, 2021
Merged

Feature/vue issue158 #211

24 commits merged into from
Mar 31, 2021

Conversation

ghost
Copy link

@ghost ghost commented Mar 26, 2021 •
edited by ghost
Loading

Jhipster 7.0.0 upgrade with vue support:

  • jwt/oauth2 security tested with all client frameworks (angular, react and vue)
  • JDL integration tests (on pipeline) with sql, mongodb, all clients and security
  • Fixed unit test and e2e tests for client and server according jhipster 7.0.0 upgrade (user controller changed on rest api endpoint, that is for admin or public)

This Nhipster blueprint upgrade has required a lot of stuff.

Angelo Manganiello added 24 commits March 4, 2021 17:50
try to migrate jhipster 7
4630552
need check client template part
0ed100a
need fix entity generator
4e837a7
try fix test error
0285061
merge from release 1.5.0
13edd59
fix all new template, need check i18n
83c0706
fix languages and skipUserManagement
77c0f74
fix entity generator
5604b0c
fix relationships but need fix import-jdl
5026c14
delete id from entity gen
2271ff3
migrate jhipster 7.0.0
8b5a08a
try fix tests
77eb4bb
update yeoman-test to fix tests
5cd4134
fix copy dist changed for client
74ed760
added admin controller for react but need check update create user an…
18598ea 
…d update entity
need check create and update usermanagement react api
19b504b
added entity client for react and reorder public api user
8836403
fix react new user management error
b24a3de
added vuejs templates
34b7ac8
add integration test for vue
35325ee
try fix error for change settings
0213ee4
fix vue oauth2
621f72f
fix mongodb update
4b6cc8a
fix account service unit vue oauth2 test
e7a85ad
@ghost ghost requested a review from mraible March 26, 2021 14:55
@ghost ghost self-assigned this Mar 26, 2021
This was linked to issues Mar 26, 2021
Vue.js support from 7.x jhipster upgrade #158
Closed
Unable to create a new entity for jhipster 6.8.0 bug #147
Closed
@ghost
Copy link
Author

ghost commented Mar 28, 2021

@mraible any feedbacks?

@mraible
Copy link
Contributor

mraible commented Mar 28, 2021

I haven't had a chance to look yet. Maybe tomorrow. In the meantime, maybe other @jhipster/hipsters can help?

This was referenced Mar 29, 2021
Closed
Open
@ghost
Copy link
Author

ghost commented Mar 31, 2021 •
edited by ghost
Loading

Any news? I have tested locally and with test-integration (github actions) for the required combinations. The e2e tests generated for server and client work fine. So if there aren't feedback, could I consider the feature completed? Thanks

@mraible
Copy link
Contributor

mraible commented Mar 31, 2021

If it works, ship it! 🚢

@ghost ghost merged commit 175bcf2 into main Mar 31, 2021
@ghost ghost deleted the feature/vue-issue158 branch March 31, 2021 17:09
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mraible mraible Awaiting requested review from mraible

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Vue.js support from 7.x jhipster upgrade Unable to create a new entity for jhipster 6.8.0 bug
1 participant
@mraible