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

Stripe: Fix header for test account deletion call #4612

Merged
merged 1 commit into from
Oct 21, 2022

Conversation

curiousepic
Copy link
Contributor

Previously using Bearer Authorization instead of Basic. Previously, the call did not seem to result in actual deletions, whereas after the switch there were multiple deletions.

@curiousepic curiousepic requested a review from a team October 21, 2022 14:27
@curiousepic curiousepic force-pushed the stripe_test_account_deletion_fix branch from 0f12b63 to a24da7b Compare October 21, 2022 14:28
Previously using Bearer Authorization instead of Basic. Previously, the
call did not seem to result in actual deletions, whereas after the
switch there were multiple deletions.
@curiousepic curiousepic force-pushed the stripe_test_account_deletion_fix branch from a24da7b to 4261edb Compare October 21, 2022 14:29
Copy link
Contributor

@aenand aenand left a comment

Choose a reason for hiding this comment

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

LGTM

@curiousepic curiousepic merged commit 4261edb into master Oct 21, 2022
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