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

📝(donate) document Stripe CLI login #3085

Merged
merged 1 commit into from
Jan 11, 2024
Merged

📝(donate) document Stripe CLI login #3085

merged 1 commit into from
Jan 11, 2024

Conversation

mlbrgl
Copy link
Member

@mlbrgl mlbrgl commented Jan 10, 2024

Add instructions to log into the Stripe CLI with the same STRIPE_API_KEY env variable defined in .dev.vars. This allows testers without access to the stripe dashboard to register a local webhook and test the donation/thank-you route.

Clouflare env variables (production and preview) updated accordingly.

Preview on https://donate.owid.pages.dev updated.

@mlbrgl mlbrgl force-pushed the cloudflare-functions-sentry branch from 5c3b993 to 4a7c954 Compare January 11, 2024 15:01
@mlbrgl mlbrgl force-pushed the cloudflare-functions-sentry branch from 4a7c954 to 2de67c3 Compare January 11, 2024 15:53
@mlbrgl
Copy link
Member Author

mlbrgl commented Jan 11, 2024

Merge activity

  • Jan 11, 12:29 PM: @mlbrgl started a stack merge that includes this pull request via Graphite.
  • Jan 11, 12:43 PM: Graphite rebased this pull request as part of a merge.
  • Jan 11, 12:44 PM: @mlbrgl merged this pull request with Graphite.

@mlbrgl mlbrgl force-pushed the cloudflare-functions-sentry branch from 2de67c3 to 7b34197 Compare January 11, 2024 17:39
Base automatically changed from cloudflare-functions-sentry to master January 11, 2024 17:40
@mlbrgl mlbrgl merged commit 62cd1bb into master Jan 11, 2024
12 of 14 checks passed
@mlbrgl mlbrgl deleted the stripe-api-key branch January 11, 2024 17:44
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.

3 participants