Actions
Bug #6430
closedMigrate Google Spreadsheets code from OAuth1 to OAuth2
Story points:
0.5
Description
Presumably OAuth1's disappearance explains why no survey responses have been imported since Feb. 2015.
Updated by Tom Clegg over 9 years ago
- Subject changed from Surveys haven't been updated since Feb. 2015 to Migrate Google Spreadsheets code from OAuth1 to OAuth2
- Description updated (diff)
- Category set to Third party integration
- Story points set to 1.0
Updated by Tom Clegg over 9 years ago
- Assigned To set to Tom Clegg
- Target version set to 2015-07-08 sprint
Updated by Tom Clegg over 9 years ago
Notes on 6430-surveys-oauth2 @ 3900f65
- It's possible to migrate existing grants from OAuth1 to OAuth2, but I didn't do that. AFAIK it would be more efficient for all affected users (researchers) to revoke and re-authorize. But if it turns out someone does need this, it can be added.
- I didn't tackle the more generic GoogleSpreadsheet functionality here -- just the surveys -- but fixing the generic spreadsheets code shouldn't take much, now that the OAuth2 code is in place and there's the GoogleSurvey example to work from.
- Instructions for setting up the OAuth service are already on the wiki (Set up an OAuth service).
- I made some minor changes to the OpenHumans code in order to reuse the OAuth2 code that was already in place for OH. It could still stand to be cleaned up a bit more to simplify the admin's OAuth Service setup form, but I didn't want to turn this into an OH/OAuth refactoring story.
Updated by Brett Smith over 9 years ago
- Target version changed from 2015-07-08 sprint to 2015-07-22 sprint
Updated by Ward Vandewege over 9 years ago
- Status changed from In Progress to Resolved
- % Done changed from 50 to 100
Applied in changeset 8b5b3ada6449ba88d84d0eb7cad6e4b4f43233c8.
Updated by Tom Clegg over 9 years ago
- Status changed from Resolved to In Progress
Updated by Tom Clegg over 9 years ago
- Target version changed from 2015-07-22 sprint to 2015-08-05 sprint
- Story points changed from 0.5 to 0.0
Updated by Tom Clegg over 9 years ago
- Target version changed from 2015-08-05 sprint to 2015-08-19 sprint
- Story points changed from 0.0 to 0.5
Updated by Tom Clegg over 9 years ago
- Assigned To changed from Tom Clegg to Ward Vandewege
Assigning to Ward because I expect most of the remaining work to be review+deploy.
Updated by Ward Vandewege over 9 years ago
I reviewed df5741c4a03a9945e0c375c3705b1efb5d0ab1fa and c9a8ec6294013ac2299150ced4d6793f4dfb386d, this LGTM.
Updated by Ward Vandewege over 9 years ago
- Status changed from In Progress to Resolved
Updated by Tom Clegg over 9 years ago
- Status changed from Resolved to In Progress
Reopening because there's still (non-survey-related) spreadsheet code that needs to be updated.
Updated by Tom Clegg over 9 years ago
- Assigned To changed from Ward Vandewege to Tom Clegg
Updated by Brett Smith over 9 years ago
- Target version changed from 2015-08-19 sprint to 2015-09-02 sprint
Updated by Brett Smith over 9 years ago
- Target version changed from 2015-09-02 sprint to 2015-09-16 sprint
Updated by Ward Vandewege over 9 years ago
reviewing 6430-spreadsheets-oauth2 24a9314: tests pass, code looks good, LGTM.
Updated by Tom Clegg over 9 years ago
- Status changed from In Progress to Resolved
- % Done changed from 75 to 100
Applied in changeset 8ab195a3da82202abbc0248030e88bb761597db0.
Updated by Ward Vandewege about 5 years ago
- Target version deleted (
2015-09-16 sprint)
Actions