Page MenuHomePhabricator

OAuth - add a little notes section for admins to remember details about external accounts
ClosedPublic

Authored by btrahan on Apr 8 2014, 11:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 17, 12:34 AM
Unknown Object (File)
Tue, Apr 16, 1:37 PM
Unknown Object (File)
Fri, Apr 5, 10:49 PM
Unknown Object (File)
Fri, Apr 5, 10:49 PM
Unknown Object (File)
Fri, Apr 5, 10:25 PM
Unknown Object (File)
Fri, Apr 5, 9:59 PM
Unknown Object (File)
Mon, Apr 1, 4:01 AM
Unknown Object (File)
Fri, Mar 29, 1:09 PM
Subscribers

Details

Summary

Fixes T4755. This also includes putting in a note that Google might ToS you to use the Google+ API. Lots of code here as there was some repeated stuff between OAuth1 and OAuth2 so I made a base OAuth with less-base OAuth1 and OAuth2 inheriting from it. The JIRA provider remains an independent mess and didn't get the notes field thing.

Test Plan

looked at providers and read pretty instructions.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan retitled this revision from to OAuth - add a little notes section for admins to remember details about external accounts.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.

This all makes sense to me. JIRA should definitely remain in its own special little world.

This revision is now accepted and ready to land.Apr 8 2014, 11:16 PM
btrahan updated this revision to Diff 20715.

Closed by commit rP2d43cf12964d (authored by @btrahan).