Skip to content

feat: add better-auth support #2142

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

feat: add better-auth support #2142

wants to merge 19 commits into from
+2,817 −80

Conversation

@HimanshuKumarDutt094
Copy link

@HimanshuKumarDutt094 HimanshuKumarDutt094 commented Aug 23, 2025 •
edited
Loading

✅ Checklist

  • I have followed every step in the contributing guide (updated 2022-10-06).
  • The PR title follows the convention we established conventional-commit
  • I performed a functional test on my final commit

Changelog

I have added support for better-auth support following #2008 and #2116.
what works:

  • postgres with drizzle/prisma
  • mysql with drizzle/prisma
  • sqlite with drizzle/prisma

Screenshots

[Screenshots]
image

I am new to this so any feedback and changes welcomed.

@changeset-bot
Copy link

changeset-bot bot commented Aug 23, 2025 •
edited
Loading

🦋 Changeset detected

Latest commit: 8b11944

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
create-t3-app Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Aug 23, 2025 •
edited
Loading

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

Project Deployment Preview Comments Updated (UTC)
create-t3-app Error Error Sep 13, 2025 2:30pm

@HimanshuKumarDutt094 HimanshuKumarDutt094 changed the title feat: add better auth partial support for trpc+app router+drizzle feat: add better auth support for trpc+app router Aug 23, 2025
@HimanshuKumarDutt094
Copy link
Author

anyone know why 15.5 fails for pages router I'm stump here, I didn't touch the pages code.

@HimanshuKumarDutt094 HimanshuKumarDutt094 changed the title feat: add better auth support for trpc+app router feat: add better-auth support Sep 11, 2025
@HimanshuKumarDutt094 HimanshuKumarDutt094 mentioned this pull request Sep 11, 2025
Open
@HimanshuKumarDutt094
Copy link
Author

need a way to fix the 512 e2e matrix combo

@kitety
Copy link

kitety commented Sep 12, 2025

awesome work!

@HimanshuKumarDutt094
Copy link
Author

holy finally passed all test now

@HimanshuKumarDutt094
Copy link
Author

@juliusmarminge @nexxeln @c-ehrlich
please take a look at this pr if you folks get a chance

@raulcanodev
Copy link

Please! I need this PR to be merged, Better auth with t3 is a great improvement

@HimanshuKumarDutt094
Copy link
Author

Please! I need this PR to be merged, Better auth with t3 is a great improvement

Until merged you can copy over the schema + config. Or clone my fork + build locally and npm / pnpm / bun link.
The og next auth uses discord auth but I added GitHub one (only provider change is needed).

@HimanshuKumarDutt094 HimanshuKumarDutt094 mentioned this pull request Oct 2, 2025
Open
@HimanshuKumarDutt094
Copy link
Author

note: currently there are no docs for this. because docs themselves need overhaul and there are a lot of discussion and feature request going on regarding them.

@HimanshuKumarDutt094 HimanshuKumarDutt094 mentioned this pull request Oct 7, 2025
Open
juliusmarminge
juliusmarminge reviewed Oct 11, 2025
View reviewed changes
Copy link
Member

@juliusmarminge juliusmarminge 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!

@rodriguezaitor
Copy link

Any plan to merge this?

@rodriguezaitor
Copy link

guess never

@kitety
Copy link

kitety commented Oct 29, 2025 via email

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

Reviewers

@juliusmarminge juliusmarminge juliusmarminge left review comments

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

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

@c-ehrlich c-ehrlich Awaiting requested review from c-ehrlich c-ehrlich is a code owner

+1 more reviewer

@raulcanodev raulcanodev raulcanodev approved these changes

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.

5 participants

@HimanshuKumarDutt094 @kitety @raulcanodev @rodriguezaitor @juliusmarminge