Skip to content

[SVCS-269] Add readonly provider for OneDrive [skip-ci]#7042

Closed
felliott wants to merge 64 commits intoCenterForOpenScience:develop-backupfrom
felliott:feature/onedrive
Closed

[SVCS-269] Add readonly provider for OneDrive [skip-ci]#7042
felliott wants to merge 64 commits intoCenterForOpenScience:develop-backupfrom
felliott:feature/onedrive

Conversation

@felliott
Copy link
Member

@felliott felliott commented Mar 30, 2017

Purpose

Add support for a read-only OneDrive provider to the OSF.

Changes

Backs out write support in 178cd42.

Side effects

Will need to be Django-fied.

Ticket

[#SVCS-269] - JIRA

Entangled with WB PR: CenterForOpenScience/waterbutler#205

Based off work done by @caseyrygt, @dsterlaceGT, and @alexandr-melnikov-dev-pro in PRs #6589 and #4681.

dsterlaceGT and others added 29 commits August 16, 2017 15:33
…todo:refresh access token and get folder list
caseyrygt and others added 24 commits August 16, 2017 15:33
…Error messages - BuildError: ('JSONRenderer__onedrive_oauth_start_user', {}, None)
 * these are already handled in the base class (ExternalAccount)
 * onedrive_id => folder_id, folder_id => folder_path,
   folder_name is computed from folder_path
 * remove OneDriveNodeSettings.has_auth
   * not sure why, but it was recommended

 * let ODNodeSettings.clear_auth unset the user settings

 * add clear_settings to ODNodeSettings

 * remove ODNodeSettings.set_user_auth as unneeded, it duplicates
   .set_auth()
 * remove methods with base implementation from OD serializer

 * make addon_short_name a class variable instead of a property

 * factor out custom code from serialize_settings and use base instead

 * move custom code into credentials_are_valid and serialized_folder
 * remove unused share emails view

 * repoint routes to new views
 * copy GDrive's creds_are_valid code

 * update view names passed to api_url_for
 * It didn't need to stand alone.
 * make root folder id configurable

 * enable externalView

 * finish proper handling of folder picker

 * add more method docs with API doc links
 * Being unexportable means they cannot be archived, downloaded,
   viewed, or copied.
@felliott felliott changed the title [WIP] [SVCS-269] Add readonly provider for OneDrive [skip-ci] [SVCS-269] Add readonly provider for OneDrive [skip-ci] Aug 29, 2017
@mfraezz
Copy link
Member

mfraezz commented Sep 5, 2017

Closing in favor of #7653

@mfraezz mfraezz closed this Sep 5, 2017
@felliott felliott deleted the feature/onedrive branch November 21, 2017 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants