-
Notifications
You must be signed in to change notification settings - Fork 454
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
feat: Google Sheets - List sheets from spreadsheet #2346
feat: Google Sheets - List sheets from spreadsheet #2346
Conversation
Hi @FlorentLvr , I changed the flow to console based, as the browser-based auth and redirection is not capturing the redirection as intended. Here is the demo result: |
@MSaiKiran9, I am not sure to understand but could you please rework this notebook to be presented the same as the "Create new sheet" using a service account? This will be easier to understand if we use the same method 🙏 let me know if you have any doubts. |
@FlorentLvr Above code is using Oauth2 for auth and I just changed the browser based auth & redirection to console based, it will ask for key to authenticate in the notebook (console based), after user confirms to trust the source in a new tab. But sure, I can make it consistent with the existing templates. |
@FlorentLvr done |
@MSaiKiran9, amazing! It's working perfectly :) |
The template is now available on the master branch on this link: |
Thank you for your contribution @FlorentLvr, your PR has been merged into the master branch of awesome-notebook. |
This PR resolves #2345