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 return url #3652

Merged
merged 2 commits into from Aug 10, 2023
Merged

Stripe return url #3652

merged 2 commits into from Aug 10, 2023

Conversation

bxy4543
Copy link
Member

@bxy4543 bxy4543 commented Aug 9, 2023

πŸ€– Generated by Copilot at 00b0dc0

Summary

πŸ’΅πŸ› οΈβ™»οΈ

This pull request enhances the Stripe payment feature by allowing different currencies and custom URLs for payment outcomes. It also improves the code quality of stripe.go by refactoring some variables and constants.

stripe.go changes
currencies and URLs
autumn leaves fall fast

Walkthrough

  • Configure Stripe payment with environment variables (link, link)

@sealos-ci-robot
Copy link
Member

πŸ€– Generated by lychee action

Summary

Status Count
πŸ” Total 885
βœ… Successful 348
⏳ Timeouts 0
πŸ”€ Redirected 0
πŸ‘» Excluded 536
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

@codecov
Copy link

codecov bot commented Aug 9, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (81609b2) 67.92% compared to head (00b0dc0) 67.92%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3652   +/-   ##
=======================================
  Coverage   67.92%   67.92%           
=======================================
  Files           8        8           
  Lines         664      664           
=======================================
  Hits          451      451           
  Misses        171      171           
  Partials       42       42           

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

@lingdie lingdie merged commit 2fb77c2 into labring:main Aug 10, 2023
77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants