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

add vue jwt module #11047

Merged
merged 20 commits into from
Oct 9, 2024
Merged

Conversation

renanfranca
Copy link
Contributor

@renanfranca renanfranca commented Oct 4, 2024

See #897 (roadmap at description)

Captura de tela 2024-10-04 120041

TO-DO

  • Refactor the vue-oauth2-keycloak module template files structure like the vue-jwt module
  • Rename documentation generator/client/vue/documentation/vue-authentication-components.md to vue-oauth2-keycloak-authentication-components.md and change it's title too
  • Add vue-jwt module in the generate.sh at vuejwtapp
    • update the matrix with these new apps in github actions (I can't update the matrix before merge the generate.sh with the new vuejwtapp and vueoauth2app. The reason is that there is a step which tries to generate these two new apps using the generate.sh from the main branch)
    • keep the vueapp until merge the generate.sh with the vuejwtapp and vueoauth2app, because vueapp still be used by matrix.

…and vue-jwt module with the feature vue-authentication
…n the vue-jwt-authentication-components.md file
…ule is always correct by converting the vue-jwt-authentication-components.md into a template, vue-jwt-authentication-components.md.mustache, and using template context variables
@renanfranca renanfranca marked this pull request as ready for review October 7, 2024 12:32
@renanfranca
Copy link
Contributor Author

I put a lot of effort into making sure this module is helpful for people already familiar with Vue and hexagonal architecture. I also made sure to include some guidance for beginners using the vue-jwt module in the jhipster-lite project. The documentation walks them through how to use it with examples, while also encouraging them to explore other modules.

@renanfranca renanfranca merged commit ec0a02e into jhipster:main Oct 9, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants