feat(adapters): add remult adapter
โ๏ธ Reasoning
Adding a new adapter @auth/remult-adapter
๐งข Checklist
- [x] Documentation
- [x] Tests
- [x] Ready to be merged
- [x] @noam-honig I would like you to check it before adding "ready"
๐ซ Affected issues
๐ Resources
The latest updates on your projects. Learn more about Vercel for Git โ๏ธ
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| auth-docs | โ Ready (Inspect) | Visit Preview | ๐ฌ Add feedback | Oct 18, 2024 5:22pm |
1 Skipped Deployment
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| next-auth-docs | โฌ๏ธ Ignored (Inspect) | Visit Preview | Oct 18, 2024 5:22pm |
@jycouet is attempting to deploy a commit to the authjs Team on Vercel.
A member of the Team first needs to authorize it.
Broken Link Checker
0 broken links found. Links organised below by source page, or page where they were found.
hi @ubbe-xyz @ndom91 @ThangHuuVu @balazsorban44 Please let me know if you need anything else to proceed.
As the creator of remult - this looks great, can't wait for this to get merged
As the creator of remult - this looks great, can't wait for this to get merged
+1
Codecov Report
Attention: Patch coverage is 96.89579% with 14 lines in your changes missing coverage. Please review.
Project coverage is 40.13%. Comparing base (
e91073f) to head (749cf8f). Report is 21 commits behind head on main.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| packages/adapter-remult/src/index.ts | 95.30% | 14 Missing :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## main #11762 +/- ##
==========================================
+ Coverage 39.05% 40.13% +1.07%
==========================================
Files 191 193 +2
Lines 29994 30303 +309
Branches 1294 1355 +61
==========================================
+ Hits 11715 12162 +447
+ Misses 18279 18141 -138
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I updated the PR to used an existing container & pulled main to resolve conflics. Could you run tests again please? Thx
@balazsorban44 could you advice why we get:
Error response from daemon: No such container: next-auth-postgres-test
I'm using this in prod on my side. It would be good to finish the effort to merge it here as well ๐
Let me know,
Tadaaa ๐ฅณ All green, I guess that @balazsorban44 you would need to allow Vercel to be 100%
Then ready to merge? :)
@balazsorban44 we had a few users requesting it. Could you have a look and validate this adapter ? :) Thank you
@balazsorban44 we need this please. We will appreciate your looking into and authorising the merge. Thanks
@balazsorban44 should I resolve conflicts ?
Authjs team, any updates on this? Anything we can do to help?