Project

General

Profile

Actions

Feature #4766

closed

Upgrade oauth1 code for google doc/spreadsheet sync to oauth2

Added by Ward Vandewege over 9 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Phil Hodgson
Category:
-
Story points:
2.0

Subtasks 4 (1 open3 closed)

Task #4983: Repurpose current OAuth1 controllers and models for OAuth2ResolvedPhil Hodgson01/15/2015Actions
Task #4984: TestsResolvedPhil Hodgson01/15/2015Actions
Task #4985: Update wiki documentationResolvedPhil Hodgson01/15/2015Actions
Task #5229: Review 4766NewWard Vandewege02/17/2015Actions

Related issues

Related to Tapestry - Idea #4980: Users can connect their Tapestry account to Open Humans via OAuth2ResolvedPhil Hodgson01/15/2015Actions
Related to Tapestry - Bug #6430: Migrate Google Spreadsheets code from OAuth1 to OAuth2ResolvedTom Clegg06/25/2015Actions
Actions #1

Updated by Phil Hodgson about 9 years ago

  • Assigned To set to Phil Hodgson
Actions #2

Updated by Phil Hodgson about 9 years ago

Okay so I've committed in this branch what I believe to be enough work to get the GoogleSurveysController to synchronize a survey set up by a researcher to read data from a specified spreadsheet made available via Google's Oauth2. Now I am just wondering what is up with the GoogleSpreadsheet model and corresponding controller... What are these used for?

Actions #3

Updated by Phil Hodgson about 9 years ago

My latest commit attempts to make the GoogleSpreadsheetController work according to my best guess. I was able to get it to do stuff, and now it reports errors a bit more consistently too.

Actions #4

Updated by Phil Hodgson about 9 years ago

I've made some controller tests similar to what I'd done for Open Humans, but proper testing of the google surveys and spreadsheets stuff is going to take some doing: there's nothing existant to upgrade to Oauth2, so they'll have to be conceived from scratch. Do we want to include as part of this story some kind of model testing of google survey and spreadsheet synchronization and so on, which actually tries to simulate a spreadsheet with data?

Actions #6

Updated by Ward Vandewege about 9 years ago

  • Status changed from New to In Progress
Actions #7

Updated by Ward Vandewege over 8 years ago

  • Status changed from In Progress to Resolved

This has now been implemented as part of #6430, because we didn't notice this ticked/branch were outstanding :/

Actions

Also available in: Atom PDF