-
Notifications
You must be signed in to change notification settings - Fork 481
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
feat(nuxt): Add Nuxt SDK docs #1728
feat(nuxt): Add Nuxt SDK docs #1728
Conversation
Hey, here’s your docs preview: https://clerk.com/docs/pr/1728 |
0d59d11
to
162d9f2
Compare
32fa402
to
7954eae
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @wobsoriano, here's my update!
docs/quickstarts/nuxt.mdx
- update quickstart
- fix
<TutorialHero/>
steps, links, headings - rewrite and restructure sections
docs/references/backend/overview.mdx
- fix code file path and function name
docs/references/nuxt/clerk-middleware.mdx
- rewrite title, description, intro, sections
- rewrite and restructure to protecting routes via authentication and authorization status
- fix code logic, file paths, comments
docs/references/nuxt/overview.mdx
- rewrite title, description, intro, headings, sections
docs/references/nuxt/protecting-pages.mdx
- rewrite title, description, intro, headings, sections
docs/references/nuxt/read-session-data.mdx
- rewrite title, description, intro, headings, sections
- restructure sections
- fix code logic, file paths, comments
// Check if the user is trying to access a protected route | ||
if (isAdminRoute) { | ||
// Check if the user has the required admin role | ||
if (orgRole !== 'org:admin') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can also use the role
property here @alexisintech? Something lik,e !has({ role: 'org:admin' })
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
## Use `useAuth()` | ||
|
||
<Include src="_partials/nuxt/use-auth" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TIL 👀
Important
🔎 Previews:
clerkMiddleware
usage in Nuxt)Note:
I think it's best to review this PR after the Vue SDK PRs!
Explanation:
0.0.1
releasedThis PR:
Checklist
Resolves ECO-244