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

docs: add recipe for session and authentication #27287

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Conversation

Smef
Copy link
Contributor

@Smef Smef commented May 21, 2024

This PR adds a recipe example for managing user sessions using nuxt-auth-utils.

Database queries are managed with Drizzle, which I'm trying for the first time here. I'm not entirely sold on this after using it, though, and could probably rewrite this with something like Sequelize or TypeORM if that would be preferred.

I've also posted a more robust example repo. Would it be good to link to this? This could be updated to use Sequelize/TypeORM as well.

Copy link

stackblitz bot commented May 21, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@Smef Smef changed the title docs: Add recipe for session and authentication docs: add recipe for session and authentication May 21, 2024
@danielroe danielroe requested a review from atinux May 21, 2024 08:18
Smef and others added 2 commits May 22, 2024 19:39
@Smef
Copy link
Contributor Author

Smef commented May 22, 2024

Should we add a link to the downloadable and runnable demo implementation? https://github.com/gearbox-solutions/nuxt-auth-example

@Smef
Copy link
Contributor Author

Smef commented Jun 5, 2024

I've made some improvements to the login and registration processes on both the client and server side to work in a more SPA way without a full reload.

Copy link
Member

atinux commented Jun 10, 2024

I did not forget it is still in my todo list!

@Smef
Copy link
Contributor Author

Smef commented Jun 10, 2024

Well good news! I've just pushed some improvements, so you didn't miss anything.

Copy link
Contributor

@GalacticHypernova GalacticHypernova left a comment

Choose a reason for hiding this comment

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

Great work!
Just a few things that caught my attention

docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
docs/2.guide/4.recipes/4.sessions-and-authentication.md Outdated Show resolved Hide resolved
@Smef Smef requested a review from danielroe as a code owner July 15, 2024 16:10
@danielroe danielroe requested a review from atinux July 15, 2024 16:25
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.

4 participants