-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
feat(providers): refactor Azure B2C provider #2862
Conversation
🎉 Experimental release published on npm! npm i next-auth@0.0.0-pr.2862.59712c27 yarn add next-auth@0.0.0-pr.2862.59712c27 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Codecov Report
@@ Coverage Diff @@
## beta #2862 +/- ##
==========================================
+ Coverage 14.02% 14.09% +0.07%
==========================================
Files 85 85
Lines 1362 1355 -7
Branches 388 385 -3
==========================================
Hits 191 191
+ Misses 1106 1102 -4
+ Partials 65 62 -3
Continue to review full report at Codecov.
|
Pulled the latest & it worked great on my end. Will keep you posted if I discover anything useful re: the profile images. |
* fix(providers): refactor AzureB2C provider * chore(dev): add Azure B2C to the dev app * chore(providers): remove unnecessary config
Thanks for helping out with testing this, @BenjaminWFox!
Fixed
azure-ad-b2c
provider config to support v4.#2524,
Fixes #2735
Proof after signing in:
NOTE: We did not manage to set an image, but I added a comment so we can fix it in the future.