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

Does the model's Google sheet really require authentication? #107

Open
turbomam opened this issue Sep 10, 2021 · 1 comment
Open

Does the model's Google sheet really require authentication? #107

turbomam opened this issue Sep 10, 2021 · 1 comment

Comments

@turbomam
Copy link
Member

I think setting up the Google API authentication is one of the hardest parts of using this repo. Chris just asked me if I knew for sure that the sheet was protected, and if it is, whether it needed to be.

@gaurav
Copy link
Collaborator

gaurav commented Sep 13, 2021

The sheet permissions are pretty permissive (anyone with the link can view it), so yes, we could potentially download the sheet by using the following URL: https://docs.google.com/spreadsheets/d/1oWS7cao-fgz2MKWtyr8h2dEL9unX__0bJrWKv6mQmM4/export?format=xlsx

(Note that this does give an error when trying to open it in Excel on my computer.)

We could add this as something to do as part of #17. I'm not too concerned about the usability of sheet2linkml for now, since it's only really intended for us on the Tools team to use.

One other thing that might help with the usability in the long term is if we create a Google App ID for sheet2linkml -- if we do this right, I think users will only need to do the final step of the authentication process (i.e. clicking on the presented link and then pasting the generated authentication code back into the command line too). However, I'm not sure what this requires, and what it will cost, and if Google would need to vet sheet2linkml to make this happen. So maybe we should look into this once we've separated sheet2linkml into its own repo (#100)? Even then, I don't think it's a very high CCDH priority for now.

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

No branches or pull requests

2 participants