Page MenuHomePhabricator

In "External Accounts", replace hard-to-find tiny "link" icon with a nice button with text on it

Authored by epriestley on Feb 6 2019, 3:26 PM.



Ref T6703. Replaces the small "link" icon with a more obvious "Link External Account" button.

Moves us toward operating against $config objects instead of against $provider objects, which is more modern and will some day allow us to resolve T6703.

Test Plan

Viewed page, saw a more obvious button. Linked an external account.

Diff Detail

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

Event Timeline

epriestley created this revision.Feb 6 2019, 3:26 PM
epriestley requested review of this revision.Feb 6 2019, 3:28 PM
amckinley accepted this revision.Feb 6 2019, 5:41 PM
amckinley added inline comments.

"have a linked"

This revision is now accepted and ready to land.Feb 6 2019, 5:41 PM
epriestley updated this revision to Diff 48024.Feb 7 2019, 12:05 AM
  • Typo fix.
This revision was automatically updated to reflect the committed changes.