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

chore(docs): Fix syntax error in RBAC guide #12733

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

Conversation

benhovinga
Copy link
Contributor

@benhovinga benhovinga commented Mar 5, 2025

☕️ Reasoning

Moved the Ellipsis ( ... ) onto a separate line as a comment. The Ellipsis was throwing "Expression Expected" error. I assume this was used to indicate to the developer that more properties can be added here. Moving it inside of a comment performs the same function without throwing any errors.

Before

export const { handlers, auth } = NextAuth({
  providers: [
    Google({
      profile(profile) {
        return { role: profile.role ?? "user", ... }
      },
    })
  ],
})

After

export const { handlers, auth } = NextAuth({
  providers: [
    Google({
      profile(profile) {
        return {
          role: profile.role ?? "user",
          // ...
        }
      },
    })
  ],
})

This fix has been applied to all frameworks.

🧢 Checklist

  • Documentation
  • Tests - N/A
  • Ready to be merged

🎫 Affected issues

Fixes: #12462

📌 Resources

@benhovinga benhovinga requested a review from ndom91 as a code owner March 5, 2025 14:35
Copy link

vercel bot commented Mar 5, 2025

@benhovinga is attempting to deploy a commit to the authjs Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Mar 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
auth-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 14, 2025 0:06am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
next-auth-docs ⬜️ Ignored (Inspect) Visit Preview Mar 14, 2025 0:06am

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.

Improve Role Based Access Control Page in Guides Section
1 participant