Page MenuHomePhabricator

Add an "ExternalAccountIdentifier" table
ClosedPublic

Authored by epriestley on Feb 20 2020, 10:21 PM.
Tags
None
Referenced Files
F15436404: D21011.id50074.diff
Tue, Mar 25, 1:54 PM
F15435407: D21011.id50055.diff
Tue, Mar 25, 8:04 AM
F15433553: D21011.diff
Mon, Mar 24, 11:45 PM
F15431303: D21011.id.diff
Mon, Mar 24, 11:40 AM
F15430984: D21011.diff
Mon, Mar 24, 9:57 AM
F15425637: D21011.diff
Sun, Mar 23, 5:13 AM
F15422231: D21011.id.diff
Sat, Mar 22, 6:47 AM
F15419999: D21011.id50074.diff
Fri, Mar 21, 10:30 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.