Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • Field Day Field Day
  • Project information
    • Project information
    • Activity
    • Labels
    • Planning hierarchy
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 15
    • Issues 15
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Field Science
  • Field DayField Day
  • Merge requests
  • !13

Merged
Created Aug 23, 2019 by Craig Earley@craigOwner

Fix code maturity, primarily our approach to Google Drive

  • Overview 0
  • Commits 3
  • Changes 5

This merge request will replace our now-obsolete Google Drive API calls with a simple call to open the Google Drive app. I can't find anything in writing to confirm this, but Drive appears to be a non-standard standard app on Android devices, and anyone doing field science using Google Drive would be well-advised (even without this merge request) to keep their data locally as well as in "the cloud".

I want to emphasize (for posterity) that there are real tradeoffs to this approach: we deliberately send users out of our own app into another over which we have no control, but in return we are no longer maintaining a lot of local Drive API calls and can focus field science development time on issues within the app itself. On balance I think that's the correct tradeoff.

Another commit in this merge fixes the other "Code maturity" warning from Android, which involved hiding the refresh icon when not scanning for devices. (I'm not sure this feature is currently visible.)

EDIT: This request addresses #29 (closed) ; clearer title.

Edited Aug 23, 2019 by Craig Earley
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: fix-code-maturity