Page MenuHomePhabricator

Add an "ExternalAccountIdentifier" table
ClosedPublic

Authored by epriestley on Feb 20 2020, 10:21 PM.
Tags
None
Referenced Files
F13222541: D21011.diff
Sun, May 19, 3:44 AM
F13214544: D21011.id.diff
Fri, May 17, 11:18 AM
F13205212: D21011.diff
Wed, May 15, 1:35 AM
F13188585: D21011.diff
Sat, May 11, 5:21 AM
F13183008: D21011.id.diff
Fri, May 10, 6:03 AM
Unknown Object (File)
Tue, May 7, 8:36 AM
Unknown Object (File)
Fri, May 3, 8:37 AM
Unknown Object (File)
Fri, May 3, 6:09 AM
Subscribers
Restricted Owners Package

Details

Summary

Depends on D21010. Ref T13493. External accounts may have multiple different unique identifiers, most often when v1 of the API makes a questionable choice (and provies a mutable, non-unique, or PII identifier) and v2 of the API uses an immutable, unique, random identifier.

Allow Phabricator to store multiple identifiers per external account.

Test Plan

Storage only, see followup changes.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.Feb 20 2020, 10:21 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 23 2020, 1:44 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.