Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI to share sheets #25

Closed
dhleong opened this issue Jul 21, 2018 · 4 comments
Closed

UI to share sheets #25

dhleong opened this issue Jul 21, 2018 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@dhleong
Copy link
Owner

dhleong commented Jul 21, 2018

So other people can view it

@dhleong dhleong added the enhancement New feature or request label Jul 21, 2018
@dhleong
Copy link
Owner Author

dhleong commented Jul 21, 2018

It looks like we can launch the Sharing Dialog, but even if the user has permission to the file, our API will still get a 404 because we're using the drive.file scope and our app hasn't received permission to view the file yet....

dhleong added a commit that referenced this issue Jul 21, 2018
Refs #25

In particular, we no longer show "Loading" if there was an error,
because we probably aren't. Instead, we delete the :sheet-sources
entry if it had an :err on a successful load, and when initiating
a load.

This commit also establishes the convention that if the sheet error
is an (ex-info), the associated data map is a set of flags indicating
things that might have gone wrong. Currently we support :permissions?
which means that the active user may not have permission to view the
file; the resolution of this is provider-specific---for gdrive, we
may try letting the user open the file with Picker as a way of granting
the app permission....
@dhleong
Copy link
Owner Author

dhleong commented Jul 21, 2018

The easiest solution is to probably just request the full drive permission if you want to view someone else's sheet they've shared with you, since otherwise even if you open the sheet itself via Picker or a Drive UI integration you will still have to open any extra data sources the same way to be able to view it. Hrm...

If we don't have permission to view a file, we can direct the user to request access with a link like: https://drive.google.com/file/d/<file-id>/view

@dhleong
Copy link
Owner Author

dhleong commented Aug 18, 2018

Okay! Got OAuth developer verification just now so we can use drive scope for GMs and drive.readonly for friends/fans to be able to view the sheet without having the scary "this is an unverified app" dialog. That should simplify this quite a bit!

dhleong added a commit that referenced this issue Aug 22, 2018
dhleong added a commit that referenced this issue Aug 24, 2018
* Refactor provider-specific error handling/resolution

First step toward #25

* Cleanup unused code

* Support filtering out shared sheets vs my sheets

It'd be nice to persist the preference to local storage...

* Revoke scopes on signout! from gdrive

* Fix querying in a space for which we no longer request the scope

* Prompt to request readonly scope if we can't load a sheet

Next: if we have read scope but still can't read it, it's probably
not been shared with us and we need to request that it be shared

* Add "request access to file" UX flow

* Add API wrapper for showing the Share dialog

* Add a share button for "my" sheets

* Populate :mine? externally so it doesn't get overwritten

See the NOTEs in the comments
@dhleong
Copy link
Owner Author

dhleong commented Aug 24, 2018

Closed by #54

@dhleong dhleong closed this as completed Aug 24, 2018
@dhleong dhleong self-assigned this Sep 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant