Skip to content

Updated donate widget with new campaign information #943

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

Merged
merged 3 commits into from
Aug 23, 2025

Conversation

ksen0
Copy link
Member

@ksen0 ksen0 commented Aug 22, 2025

As there is a new PF fundraising campaign, the embedded widget is updated. Will look like this:

image

Once this is merged to 2.0, can also cherry-pick to main.

@ksen0 ksen0 requested review from davepagurek and xinemata August 22, 2025 21:26
@xinemata
Copy link

Thanks Kit! It'd be nice to tie the fundraiser message to the form, but it doesn't look like Donorbox lets me add hyperlinks on the embed form. Would it make sense to replace the message on the donate page with the Software as an Act of Love fundraiser copy during the fundraiser period (Aug 25~Sep 30, 2025)?

@raclim has also started to work on translating the copy on #3578 which may be helpful.

@ksen0
Copy link
Member Author

ksen0 commented Aug 22, 2025

I've updated the message, and @raclim I'll look into updating the translations accordingly for the other languages.

@xinemata from October, 2025, though, do you think it makes sense to use the previous text again, and/or keep all/some of the new text?

@ksen0
Copy link
Member Author

ksen0 commented Aug 22, 2025

PS @davepagurek if you have a moment, any ideas on dealing with the failed checks on scrolling and frameborder? I can remove those to deploy, but it does look better this way, so if there's a way to ignore the check (safely), that would be nice.

EDIT: Resolved

@ksen0 ksen0 merged commit 3b976bb into processing:2.0 Aug 23, 2025
4 checks passed
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