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

Enable monthly recurring donations via Stripe #414

Merged
merged 13 commits into from
Feb 27, 2025
Merged

Enable monthly recurring donations via Stripe #414

merged 13 commits into from
Feb 27, 2025

Conversation

dergigi
Copy link
Member

@dergigi dergigi commented Feb 27, 2025

Build preview:


This is the simplest setup I could think of without breaking any of the existing stuff.

Note: There aren't any links to other things in the text and this is on purpose. We could very easily link to our mission and to transparency and to other things mentioned, but from a Don't Make Me Think usability perspective any additional links are distracting and will lead to drop-off.

Copy link

vercel bot commented Feb 27, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
os-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 27, 2025 5:02pm

@dergigi dergigi self-assigned this Feb 27, 2025
@dergigi dergigi requested a review from Arvin21M February 27, 2025 13:08
@dergigi dergigi marked this pull request as ready for review February 27, 2025 13:08
@dergigi dergigi marked this pull request as draft February 27, 2025 13:40
Copy link
Contributor

@Arvin21M Arvin21M left a comment

Choose a reason for hiding this comment

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

A few relatively small change requests.
Otherwise, this is wonderful to have! 🧡
:shipit:

Co-authored-by: ARVIN <[email protected]>
@dergigi dergigi requested a review from lucasmogu February 27, 2025 16:32
@dergigi dergigi marked this pull request as ready for review February 27, 2025 16:32
@dergigi dergigi requested a review from Arvin21M February 27, 2025 16:32
Copy link
Contributor

@Arvin21M Arvin21M left a comment

Choose a reason for hiding this comment

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

  • Donate button hover color change looks good!
    • Changes from white to light/pastel orange.
  • Clicking button goes to stripe payment page.
    • Seeing "Pay with Link" at the top, I assume that is supposed to be there.
    • Another reference to Link towards the bottom of Stripe payment page.
  • Also marked one really small edit request.
  • Otherwise, good to go! 🚀

Co-authored-by: ARVIN <[email protected]>
@dergigi dergigi merged commit 6cbf61e into master Feb 27, 2025
3 checks passed
@dergigi dergigi deleted the recurring branch February 27, 2025 20:04
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