Page MenuHomePhabricator

Write ExternalAccountIdentifiers when interacting with external authentication providers
ClosedPublic

Authored by epriestley on Feb 22 2020, 4:30 PM.

Details

Summary

Depends on D21015. When we sync an external account and get a list of account identifiers, write them to the database.

Nothing reads them yet and we still write "accountId", this just prepares us for reads.

Test Plan

Linked, refreshed, unlinked, and re-linked an external account. Peeked at the database and saw a sensible-looking row.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Feb 22 2020, 4:30 PM
epriestley requested review of this revision.Feb 22 2020, 4:31 PM
epriestley updated this revision to Diff 50071.Feb 23 2020, 1:30 AM
  • Also remove "realName" when unlinking an account.
This revision was not accepted when it landed; it landed in state Needs Review.Feb 23 2020, 1:46 AM
This revision was automatically updated to reflect the committed changes.