next-auth icon indicating copy to clipboard operation
next-auth copied to clipboard

feat(adapter): add @auth/postgresjs-adapter

Open mustaqimarifin opened this issue 1 year ago โ€ข 9 comments

โ˜•๏ธ Reasoning

Adding a new adapter using Postgres.js.

๐Ÿงข Checklist

  • [X] Documentation
  • [X] Tests
  • [X] Ready to be merged

๐ŸŽซ Affected issues

๐Ÿ“Œ Resources

mustaqimarifin avatar Jun 18 '24 12:06 mustaqimarifin

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

vercel[bot] avatar Jun 18 '24 12:06 vercel[bot]

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

A member of the Team first needs to authorize it.

vercel[bot] avatar Jun 18 '24 12:06 vercel[bot]

๐Ÿ‘ 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.

View full reportโ†—๏ธŽ

socket-security[bot] avatar Jun 18 '24 12:06 socket-security[bot]

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

ndom91 avatar Jun 20 '24 13:06 ndom91

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?

mustaqimarifin avatar Jun 22 '24 20:06 mustaqimarifin

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

ndom91 avatar Jun 24 '24 18:06 ndom91

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.

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 ๐Ÿ˜Ž

mustaqimarifin avatar Jun 24 '24 23:06 mustaqimarifin

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]

View full reportโ†—๏ธŽ

socket-security[bot] avatar Jul 21 '24 09:07 socket-security[bot]

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.

codecov[bot] avatar Jul 21 '24 09:07 codecov[bot]

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!

stale[bot] avatar Jan 21 '25 23:01 stale[bot]

Oh, what a pity this PR didn't go further. It seems to me like it should be ready.

ctwhome avatar Jan 28 '25 13:01 ctwhome