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-v4): remove broken links in tutorials #12474

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

halvaradop
Copy link
Contributor

☕️ Reasoning

This pull request removes broken links in the Tutorials and Explainers section of the v4 version of next-auth. Some links were either non-existent or broken.

Key Changes

  • Authentication patterns in Next.js: The link was updated on the Next.js website to this. However, I have not updated this link because it does not showcase implementations that use next-auth v4. The current documentation on Vercel covers usages of the beta version, which does not align with the target of the links in the v4 version of the documentation, such as in this link.
  • Adding social authentication support to a Next.js app: This link is broken.
  • Fullstack Authentication Example with Next.js and NextAuth.js: The repository tree was updated to this, but the example content no longer implements next-auth.

Note: Some links might change, so I will be attentive to any suggestions and possible updates.

🧢 Checklist

  • Documentation
  • Tests
  • Ready to be merged

🎫 Affected issues

📌 Resources

@halvaradop halvaradop requested a review from ndom91 as a code owner January 6, 2025 17:52
Copy link

vercel bot commented Jan 6, 2025

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

Name Status Preview Comments Updated (UTC)
auth-docs ❌ Failed (Inspect) Jan 6, 2025 5:54pm
next-auth-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 6, 2025 5:54pm

Copy link

vercel bot commented Jan 6, 2025

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

A member of the Team first needs to authorize it.

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.

2 participants