Skip to content

Improve Credentials page with platform-specific integration user guidance#777

Open
hunterachieng wants to merge 4 commits into
mainfrom
feature/142-platform-creds
Open

Improve Credentials page with platform-specific integration user guidance#777
hunterachieng wants to merge 4 commits into
mainfrom
feature/142-platform-creds

Conversation

@hunterachieng
Copy link
Copy Markdown
Contributor

Short Description

Improved the Credentials page and Google Sheets/Drive adaptor docs with platform-specific integration user guidance, service account recommendations, and updated credential screenshots.

Closes #142

Details

Adds platform-specific integration user guidance to the Credentials page and relevant Google adaptor pages. The Credentials page now links out to Salesforce and Google Sheets for platform-specific advice rather than duplicating content.

The Google Sheets and Google Drive adaptor pages each gain a "Using a Google Service Account" section covering least-privilege scope recommendations and resource sharing, plus updated screenshots for both OAuth and access token credential types.

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to
know!):

  • I have used Claude Code
  • I have used another model
  • I have not used AI

You can read more details in our
Responsible AI Policy

Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
Comment thread adaptors/googledrive.md Outdated
Comment thread adaptors/googledrive.md Outdated
## Authentication

Google Drive uses OAuth 2.0 access tokens for API authentication:
Google Drive supports two credential types in OpenFn: **OAuth2** and a raw **access token**.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hunterachieng this not accurate, the raw access token is the OAuth 2.0 access token, Please revert this back to

Google Drive uses OAuth 2.0 access tokens for API authentication:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mtuchi I checked how we can add credentials for Google Drive and it uses both OAuth and you can add a raw access token on the creds page

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hunterachieng yes it's true you can add raw credential with any adaptor not only googlesheet, Are you saying it's possible to create a long lived access token from google cloud console ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, the token only lives for 1 hour. Should I change and only focus on Oauth2?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hunterachieng Yes, i think the way it was before is fine, we can add a foot note saying If you need to test the adaptor locally you, you can generate a 1hour token using gcloud cli

Comment thread adaptors/googledrive.md Outdated
Copy link
Copy Markdown
Contributor

@mtuchi mtuchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @hunterachieng i left some comments in googledrive adaptor can you work on those and see if also you the other adaptor need the same changes

Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
@hunterachieng hunterachieng requested a review from mtuchi May 26, 2026 14:33
Comment thread adaptors/googledrive.md Outdated
Signed-off-by: Hunter Achieng <achienghunter@gmail.com>
@hunterachieng hunterachieng requested a review from mtuchi May 28, 2026 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants