Skip to content

linked in provider options and profile update - #8345

Merged
vuhuuthang15 merged 12 commits into
nextauthjs:mainfrom
HugeLetters:main
Aug 29, 2023
Merged

vuhuuthang15 merged 12 commits into
nextauthjs:mainfrom
HugeLetters:main

Conversation

@HugeLetters

Copy link
Copy Markdown
Contributor

☕️ Reasoning

It seems LinkedIn has updated its authentication procedure - this PR updates the provider to reflect those changes.
LinkedIn renamed required scopes(permissions) and also changed the shape of profile data.

🧢 Checklist

  • Documentation - not needed
  • Tests
  • Ready to be merged

🎫 Affected issues

Please scout and link issues that might be solved by this PR.

Fixes: #8316

📌 Resources

@vercel

vercel Bot commented Aug 17, 2023 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
auth-docs ❌ Failed (Inspect) Aug 29, 2023 4:17am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
next-auth-docs ⬜️ Ignored (Inspect) Visit Preview Aug 29, 2023 4:17am

@vercel

vercel Bot commented Aug 17, 2023

Copy link
Copy Markdown

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

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added providers core Refers to `@auth/core` legacy Refers to `next-auth` v4. Minimal maintenance. labels Aug 17, 2023
@joemulberry

Copy link
Copy Markdown

would love this to be solved

@ss-zheng

Copy link
Copy Markdown

OMG I was stucked for a night in order to enable linkedin auth, and found this PR this morning. Please merge this appreciated it!

@vuhuuthang15 vuhuuthang15 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution, I left some comments. 🙏

Comment thread packages/core/src/providers/linkedin.ts Resolved
@github-actions github-actions Bot removed the legacy Refers to `next-auth` v4. Minimal maintenance. label Aug 28, 2023
Comment thread packages/core/src/providers/linkedin.ts Resolved
@AnatoleT

Copy link
Copy Markdown

Would love to see this solved too :)

This branch had an error being deployed

1 failed deployment
Preview – auth-docs — 8d7940b5 Deployed Aug 29, 2023 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Refers to `@auth/core` providers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sign In with LinkedIn is deprecated, use Sign In with LinkedIn using OpenID Connect instead

5 participants