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 Auth0 to Spring Boot + Vue CRUD post #1284

Merged
merged 35 commits into from
Jan 20, 2023

Conversation

moksamedia
Copy link
Contributor

@moksamedia moksamedia commented Oct 10, 2022

Updates Build a Simple CRUD App with Spring Boot and Vue.js to add Auth0.

Example app changes can be viewed in oktadev/okta-spring-boot-vue-crud-example#6.

@netlify
Copy link

netlify bot commented Oct 10, 2022

Deploy Preview for okta-blog ready!

Name Link
🔨 Latest commit 4af1d05
🔍 Latest deploy log https://app.netlify.com/sites/okta-blog/deploys/63cb107eedad6c00084d6f5f
😎 Deploy Preview https://deploy-preview-1284--okta-blog.netlify.app/blog/2022/08/19/build-crud-spring-and-vue
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Contributor

@mraible mraible left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a few suggestions and questions. Back to you @moksamedia!

Copy link
Contributor

@mraible mraible left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@moksamedia I added some minor suggestions for improvement. This post still needs a changelog so people know it's been updated with Auth0 instructions. Did you update the example app with a new branch? If not, I can do it as part of QA.

@havi This one is ready for your review. This is an update to an existing post.

@mraible mraible changed the title Spring boot vue add auth0 Add Auth0 to Spring Boot + Vue CRUD post Dec 8, 2022
moksamedia and others added 7 commits December 19, 2022 13:11
Co-authored-by: Matt Raible <matt.raible@okta.com>
Co-authored-by: Matt Raible <matt.raible@okta.com>
Co-authored-by: Matt Raible <matt.raible@okta.com>
Co-authored-by: Matt Raible <matt.raible@okta.com>
Co-authored-by: Matt Raible <matt.raible@okta.com>
Co-authored-by: Matt Raible <matt.raible@okta.com>
Copy link
Contributor

@havi havi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple fussy/typo catches. And a couple updates for clarity/ease of reading

@mraible mraible merged commit 704ce2a into oktadev:main Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants