Skip to content
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

Update LinkedIn Stamp and Incorporate Email Verification #2915

Open
2 tasks
erichfi opened this issue Sep 25, 2024 · 1 comment · May be fixed by #3001
Open
2 tasks

Update LinkedIn Stamp and Incorporate Email Verification #2915

erichfi opened this issue Sep 25, 2024 · 1 comment · May be fixed by #3001
Assignees

Comments

@erichfi
Copy link
Collaborator

erichfi commented Sep 25, 2024

User Story:

As a Passport user, I want to connect my LinkedIn account using the new API (V2) so that I can verify my email address and ensure accurate profile data.

Acceptance Criteria

GIVEN a user connecting their LinkedIn account,
WHEN they use the OAuth flow,
THEN the system should retrieve both the unique user identifier and email verification status using LinkedIn API V2.
AND any LinkedIn Stamp in our DB from V1 should be removed or marked as expired/revoked during the cutover.

GIVEN a user lost their v1 Linkedin Stamp,
WHEN they enter the Passport app,
THEN they should see a targeted notification letting them that they need to re-verify their LinkedIn stamp because of a change with the LinkedIn API.

Product & Design Links:

N/A

Tech Details:

  • Update existing LinkedIn provider to utilize API V2.
  • Adjust IAM logic to check for verified email status.
  • Ensure unique user identifiers from V2 are used for future deduplication.
  • Remove or mark expired/revoked existing LinkedIn Stamps in DB during cutover.

Open Questions:

  • Any additional data points from V2 to consider?
  • Specific users or segments to inform about this change?

Notes/Assumptions:

  • Ignore historical data from LinkedIn API V1.
  • Manage the transition period, addressing potential temporary duplication.
@nutrina
Copy link
Collaborator

nutrina commented Sep 30, 2024

  • modify the current LinkedIn provider to use V2 of the API, this also means creating a new provider name LinkedinV2 for this (+ new provider for the email verification)
  • update the credentials for LinkedIn OAuth
  • we need to retire the current LinkedIn stamps (because the V2 version of the API returns a different D for the user)
    • in order to deal with the new stamp version, we might set the weight of the old LinkedIn stamp to 0, and proper weights to the new one

@larisa17 larisa17 self-assigned this Oct 14, 2024
@larisa17 larisa17 linked a pull request Oct 24, 2024 that will close this issue
@larisa17 larisa17 linked a pull request Oct 24, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress (WIP)
Development

Successfully merging a pull request may close this issue.

3 participants