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

Update companion.md #303

Closed
wants to merge 1 commit into from
Closed

Update companion.md #303

wants to merge 1 commit into from

Conversation

mifi
Copy link
Contributor

@mifi mifi commented Jan 10, 2025

No description provided.

Copy link
Member

@Murderlon Murderlon left a comment

Choose a reason for hiding this comment

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

I don't think this belong here and should instead of documented on the google-drive-picker docs page.

@mifi
Copy link
Contributor Author

mifi commented Jan 13, 2025

When I read the text:

To do so each provider plugin must be configured with Transloadit’s Companion URLs:
...
You may also hit rate limits, because the OAuth application is shared between everyone using Transloadit.

To solve that, you can use your own OAuth keys with Transloadit’s hosted Companion servers by using Transloadit Template Credentials. Create a Template Credential on the Transloadit site. Select “Companion OAuth” for the service, and enter the key and secret for the provider you want to use. Then you can pass the name of the new credentials to that provider:

I feel it it is written in a way that it applies to all Companion plugins, and so it can be confusing because what's written doesn't apply at all to Google Drive Picker and Google Photos Picker. that's why I wanted to mention it here.

@Murderlon
Copy link
Member

My assumption is that everyone who will need a google plugin is forced to go to the google plugin docs and thus they see it. This is more a generic mention of hosted Transloadit.

@mifi mifi closed this Feb 14, 2025
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.

2 participants