Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Landing Page #13192

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

Open
TheAmanM wants to merge 22 commits into nextauthjs:main
base: main
Choose a base branch
Loading
from TheAmanM:feat/landing-page
Open

Landing Page #13192

TheAmanM wants to merge 22 commits into nextauthjs:main from TheAmanM:feat/landing-page

Conversation

Copy link

@TheAmanM TheAmanM commented Aug 26, 2025
edited
Loading

Landing Page (View here)

image

Auth JS is awesome. The landing page sucks. Hundreds of hours, if not more have gone into creating a free and open source way for developers to have an easier time integrating authentication into their applications, whether they're a Next JS developer or making an Express server.

Solution

This PR is designed to re-design and implement a new landing page. It also refactors index.mdx.

🎫 Affected issues

Addresses #13194

📌 Resources

Copy link

vercel bot commented Aug 26, 2025
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
auth-docs Ready Ready Preview Comment Aug 27, 2025 10:08am
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
next-auth-docs Ignored Ignored Preview Aug 27, 2025 10:08am

Copy link

vercel bot commented Aug 26, 2025

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

A member of the Team first needs to authorize it.

@TheAmanM TheAmanM changed the title (削除) fixed logos (削除ここまで) (追記) Landing Page (追記ここまで) Aug 26, 2025
Copy link
Author

Alright, I think I'm done adding features. This was challenging and fun!

halvaradop

This comment was marked as duplicate.

Copy link
Contributor

@halvaradop halvaradop 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 on this! Here are a few suggestions to consider for improvement.

Copy link
Author

Thank you for the direction - it's definitely not easy to navigate a new codebase!

I've made the desired changes:

  • Swapped to
  • Switched from a custom providers array to reusing providersOAuth
  • Changes {name}.svg to just the name as registered in manifest.json
  • Moved styles to globals.css
  • Reused RichTabs and implemented CSS
halvaradop reacted with rocket emoji

Copy link
Contributor

halvaradop commented Aug 27, 2025
edited
Loading

Thanks for taking my suggestions into account. You’ve done a great job!

Copy link
Member

ndom91 commented Aug 27, 2025

Love the effort here as well. Haven't dug very deep into the PR, but let's already remove the * around "anything" in the copy visible in the screenshot in the PR desc.

I'd say if unsure, err toward looking more professional.

Love a lot of the sections here though!

Copy link
Author

Thanks for taking my suggestions into account. You’ve done a great job!

Thank you so much! This is my first time - contributing to OSS at this scale, and I'm glad I took the initiative!

Copy link
Author

Love the effort here as well. Haven't dug very deep into the PR, but let's already remove the * around "anything" in the copy visible in the screenshot in the PR desc.

I'd say if unsure, err toward looking more professional.

Love a lot of the sections here though!

That makes sense — I found myself getting curious with how certain brands pull off a more casual style.

I've removed the asterisks, any other requests I'm able to help with?

Copy link
Author

TheAmanM commented Sep 1, 2025

Hi, just to confirm, we're waiting on @ndom91 right? Or am I missing something?

First time contributor here, so I might be missing something!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ndom91 ndom91 Awaiting requested review from ndom91 ndom91 is a code owner

1 more reviewer

@halvaradop halvaradop halvaradop left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /