django / djangoproject.com

Source code to djangoproject.com
https://www.djangoproject.com/
BSD 3-Clause "New" or "Revised" License
1.86k stars 937 forks source link

Upgraded to django-recaptcha 4.0.0 (third time's the charm 🍀 ) #1480

Closed bmispelon closed 4 months ago

bmispelon commented 4 months ago

I took the work from #1450 (which was reverted later because of https://github.com/django-recaptcha/django-recaptcha/issues/348).

For ease of review, I added an extra commit which I believe fixes the issue we encountered (the two commits can then be squashed before merging)

Here's how I tested it locally:

1) Commented out postToStripe(token) in our own stripe-donation.js 2) Loaded the page at /fundraising/ on my local version 3) Clicked the "donate monthly" button 4) Without my fix, the form is submitted and the page reloads. With my fix, the page doesn't reload and the console shows "reCAPTCHA validated. Posting to stripe...")

felixxm commented 4 months ago

Unfortunately, reverted.