Skip to content

fix(auth): include drive scope for sheets export#62

Merged
steipete merged 1 commit intomainfrom
temp/fix-58-sheets-readonly-drive
Jan 10, 2026
Merged

fix(auth): include drive scope for sheets export#62
steipete merged 1 commit intomainfrom
temp/fix-58-sheets-readonly-drive

Conversation

@steipete
Copy link
Copy Markdown
Collaborator

@steipete steipete commented Jan 10, 2026

Fixes readonly Sheets export: gog auth add --services sheets --readonly now includes https://www.googleapis.com/auth/drive.readonly, since gog sheets export goes through Drive.

  • Sheets service scopes include Drive (export via Drive)
  • Scopes selection adds Drive scope for Sheets (respects --drive-scope)
  • Regression tests for both scope selection + CLI auth add

@steipete steipete merged commit 528cba3 into main Jan 10, 2026
7 checks passed
manascb1344 pushed a commit to manascb1344/gogcli that referenced this pull request Jan 15, 2026
…nly-drive

fix(auth): include drive scope for sheets export
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.

1 participant