Page MenuHomePhabricator
Differential D21016

Write ExternalAccountIdentifiers when interacting with external authentication providers
ClosedPublic
Actions

Authored by epriestley on Feb 22 2020, 4:30 PM.
Tags
None
Referenced Files
F15434967: D21016.id.diff
Tue, Mar 25, 6:12 AM
F15429978: D21016.id.diff
Mon, Mar 24, 5:06 AM
F15429941: D21016.diff
Mon, Mar 24, 4:59 AM
F15411646: D21016.diff
Wed, Mar 19, 9:30 AM
F15394078: D21016.diff
Sat, Mar 15, 11:19 PM
F15388873: D21016.id.diff
Sat, Mar 15, 4:34 AM
F15378008: D21016.id50071.diff
Thu, Mar 13, 12:01 PM
F15377248: D21016.id50079.diff
Thu, Mar 13, 8:27 AM
View All Files
Subscribers
None

Details

Reviewers
None
Commits
rPbcaf60015a47: Write ExternalAccountIdentifiers when interacting with external authentication…
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
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

Diff 50079

View Options

src/applications/auth/controller/PhabricatorAuthUnlinkController.php

Loading...
View Options

src/applications/auth/provider/PhabricatorAuthProvider.php

Loading...
View Options

src/applications/auth/query/PhabricatorExternalAccountQuery.php

Loading...
View Options

src/applications/people/storage/PhabricatorExternalAccount.php

Loading...
View Options

src/applications/people/storage/PhabricatorExternalAccountIdentifier.php

Loading...
Phabricator · Built by Phacility