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

Source Stripe: Add more detail around stream permissions #15585

Closed
brianjlai opened this issue Aug 12, 2022 · 3 comments
Closed

Source Stripe: Add more detail around stream permissions #15585

brianjlai opened this issue Aug 12, 2022 · 3 comments
Assignees
Labels
area/documentation Improvements or additions to documentation autoteam team/connectors-python team/documentation type/enhancement New feature or request

Comments

@brianjlai
Copy link
Contributor

Tell us about the documentation you'd like us to add or update.

On Sentry we've been seeing a handful or errors where customers are configuring their Stripe connector w/ accounts that don't have enough permissions to sync certain streams. We might want to make it more clear what sets of permissions they need so they don't go through with the sync process that will end up failing.

If applicable, add links to the relevant docs that should be updated

@brianjlai brianjlai added area/documentation Improvements or additions to documentation type/enhancement New feature or request labels Aug 12, 2022
@Amruta-Ranade
Copy link
Contributor

@brianjlai Do we have a list of the permissions Stripe users need? Or is compiling the list part of this issue?

@brianjlai
Copy link
Contributor Author

@Amruta-Ranade I'm so sorry I completely missed this email notification. Yes it would be compiling the list of permissions for each stream. Stripe does do a good job of documenting which API resources correlate to each permission in their docs: https://stripe.com/docs/stripe-apps/reference/permissions#object so on our side, we need to correlate the stream back to the permission type, similar to how we do it for Hubspot

@davydov-d davydov-d self-assigned this Jul 20, 2023
@arsenlosenko arsenlosenko self-assigned this Aug 15, 2023
@arsenlosenko arsenlosenko changed the title Add more detail around Stripe permissioning Source Stripe: Add more detail around stream permissions Aug 15, 2023
@arsenlosenko
Copy link
Contributor

this functionality was recently implemented and merged with this PR:
#29309
This change adds logging warnings in case if some permissions are missing. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation autoteam team/connectors-python team/documentation type/enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants