Page MenuHomePhabricator
Differential D20113

Make external account unlinking use account IDs, not "providerType + providerDomain" nonsense
ClosedPublic
Actions

Authored by epriestley on Feb 7 2019, 12:02 AM.
Tags
None
Referenced Files
F15431664: D20113.diff
Mon, Mar 24, 1:50 PM
F15419364: D20113.id48129.diff
Fri, Mar 21, 5:36 AM
F15406908: D20113.id48129.diff
Tue, Mar 18, 2:39 PM
F15402342: D20113.id48022.diff
Mon, Mar 17, 10:03 PM
F15370142: D20113.diff
Wed, Mar 12, 9:17 AM
F15308671: D20113.diff
Thu, Mar 6, 6:32 AM
Unknown Object (File)
Feb 20 2025, 10:23 AM
Unknown Object (File)
Feb 9 2025, 7:07 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T6703: Allow multiple copies of the same auth provider type
Commits
rPe5ee656fff0c: Make external account unlinking use account IDs, not "providerType +…
Summary

Depends on D20112. Ref T6703. When you go to unlink an account, unlink it by ID. Crazy!

Test Plan

Unlinked and relinked Google accounts.

Diff Detail

Repository
rP Phabricator
Branch
welcome16
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21862
Build 29837: Run Core Tests
Build 29836: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
applications/
auth/
application/
2 lines
controller/
119 lines
settings/
panel/
2 lines

Diff 48022

View Options

src/applications/auth/application/PhabricatorAuthApplication.php

Loading...
View Options

src/applications/auth/controller/PhabricatorAuthUnlinkController.php

Loading...
View Options

src/applications/settings/panel/PhabricatorExternalAccountsSettingsPanel.php

Loading...
Phabricator · Built by Phacility