-
Notifications
You must be signed in to change notification settings - Fork 31
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
Submission error in Brave browser #34
Comments
this is usually the result of:
can you confirm it's non of the above? thx |
This is definitely an issue with the Brave browser, which aggressively blocks external API calls as part of their ad blocking. Others have reported similar issues. https://community.brave.com/t/mailchimp-website-widget-unresponsive-to-submit-button/12788 We added some (hacky) JS to detect when the error occurs and tell users to change their Brave browser setting:
|
This issue happens with firefox too, but if you embed the mailchimp form it doesnt have the issue |
Hey @mifeng, Where did you add this? This could be easily changed into something generic like
I feel this would be doing the trick for most of the users out there and avoid people leaving because it appears broken? Thanks for your help! |
@benjaminhoffman is this still on your radar? |
yes! so sorry. just been super swamped. let me see if i can get to it this weekend, but at worst, i will do before feb. |
No worries, totally understand :) Let me know if I can be of any help |
@benjaminhoffman did you check the calendar? :P |
@littlehamster fastest way to get this solved is by opening a PR! :) will do my best to get to it but got a million other things going on too. |
@benjaminhoffman right now I am just embeding the code directly |
When testing the v2 example in the Brave browser (https://brave.com/), the plugin throws an error upon submission rather than returning the Mailchimp response:
index.js
is/node_modules/jsonp/index.js
The text was updated successfully, but these errors were encountered: