feat(adapter): add @auth/postgresjs-adapter
โ๏ธ Reasoning
Adding a new adapter using Postgres.js.
๐งข Checklist
- [X] Documentation
- [X] Tests
- [X] Ready to be merged
๐ซ Affected issues
๐ Resources
The latest updates on your projects. Learn more about Vercel for Git โ๏ธ
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| auth-docs | โ Failed (Inspect) | Aug 2, 2024 3:00pm |
1 Skipped Deployment
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| next-auth-docs | โฌ๏ธ Ignored (Inspect) | Visit Preview | Aug 2, 2024 3:00pm |
Someone is attempting to deploy a commit to the authjs Team on Vercel.
A member of the Team first needs to authorize it.
๐ Dependency issues cleared. Learn more about Socket for GitHub โ๏ธ
This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.
Theres already an open PR adding an adapter using postgres.js (see: https://github.com/nextauthjs/next-auth/pull/10570).
Ideally you two could sync and work on just 1 PR together :pray:
EDIT: Looks like that other user, @CodyBontecou, closed his PR in favor of this one
Theres already an open PR adding an adapter using
postgres.js(see: #10570).Ideally you two could sync and work on just 1 PR together ๐
EDIT: Looks like that other user, @CodyBontecou, closed his PR in favor of this one
hello! sounds good. are there any pending changes i need to make?
Theres already an open PR adding an adapter using
postgres.js(see: #10570).Ideally you two could sync and work on just 1 PR together ๐
EDIT: Looks like that other user, @CodyBontecou, closed his PR in favor of this one
hello! sounds good. are there any pending changes i need to make?
Is that _test.txt file not supposed to be a shell script instead? But otherwise looks alright at first glance.
Give us some time to look over things. We're a bit swamped getting in the qwik, nuxt and other client libs atm
Theres already an open PR adding an adapter using
postgres.js(see: #10570). Ideally you two could sync and work on just 1 PR together ๐ EDIT: Looks like that other user, @CodyBontecou, closed his PR in favor of this onehello! sounds good. are there any pending changes i need to make?
Is that
_test.txtfile not supposed to be a shell script instead? But otherwise looks alright at first glance.
Thanks for catching that... sorted!
Give us some time to look over things. We're a bit swamped getting in the qwik, nuxt and other client libs atm
no worries ๐
New and removed dependencies detected. Learn more about Socket for GitHub โ๏ธ
| Package | New capabilities | Transitives | Size | Publisher |
|---|
๐ฎ Removed packages: npm/@types/[email protected], npm/@types/[email protected], npm/@typescript-eslint/[email protected], npm/[email protected], npm/[email protected]
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Please upload report for BASE (
main@089566f). Learn more about missing BASE report.
Additional details and impacted files
@@ Coverage Diff @@
## main #11200 +/- ##
=======================================
Coverage ? 97.59%
=======================================
Files ? 6
Lines ? 1289
Branches ? 142
=======================================
Hits ? 1258
Misses ? 31
Partials ? 0
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
It looks like this issue did not receive any activity for 60 days. It will be closed in 7 days if no further activity occurs. If you think your issue is still relevant, commenting will keep it open. Thanks!
Oh, what a pity this PR didn't go further. It seems to me like it should be ready.