-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
adjust default fusionauth provider details #10868
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
adjust default fusionauth provider details #10868
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
@codercatdev is attempting to deploy a commit to the authjs Team on Vercel. A member of the Team first needs to authorize it. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10868 +/- ##
==========================================
- Coverage 39.00% 38.76% -0.25%
==========================================
Files 200 200
Lines 32122 32322 +200
Branches 1400 1399 -1
==========================================
Hits 12529 12529
- Misses 19593 19793 +200 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@ThangHuuVu let me know if there is anything else that needs to be adjusted for this. I don't see a clear way to pass the codecov tests for this change. |
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! |
@falcowinkler thanks for approving, I actually have a follow up that adds in even more functionality if possible? Has more of a breakdown with refresh tokens we would expect. |
@alex-fusionauth ok, just add the changes to this PR and we will check it out :) |
@falcowinkler lmk if you need a fusionauth instance or anything to try this out on. |
okay @falcowinkler it should be all set, I don't see a way in the provider to also set the callbacks, so I just added it to the details. |
any way we can get this updated and merged into the main branch? Happy to help test or update |
@falcowinkler I have updated this again to see if we can get merged, can you authorize? |
☕️ Reasoning
Adjusting defaults for FusionAuth provider.
🧢 Checklist
🎫 Affected issues
📌 Resources