-
Notifications
You must be signed in to change notification settings - Fork 63
fix: dataSize default overriding actual config #85
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
fix: dataSize default overriding actual config #85
Conversation
tomorrow 😄👍 |
Sounds great! Thank you! |
yipp i did npm install https://github.com/andrewmurraydavid/recaptcha-module.git, but to no avail, i will wait ... |
I apologize for the late response, it's been a long day at work. I've created a temporary npm package (don't worry @mvrlin, I will remove this when we solve this issue). @mangrove108 feel free to install the temporary package I created to test the changes: npm i @andrewmurraydavid/recaptcha@1.0.4-alpha.1 You will also need to change the module in nuxt.config.js -'@nuxtjs/recaptcha',
+'@andrewmurraydavid/recaptcha', Feel free to add me on discord (see my profile). More than happy to take the time and do a paired session to see exactly what's breaking on your side and get it fixed. |
Just opened, and then closed, issue #88 as this pull request seems to fix the error with recaptcha showing. |
@mangrove108 did you get a chance to test the changes? @Risen-Solutions could you checkout and try to test this solution? Thanks! |
Hi All. I can confirm that I have checked out "@andrewmurraydavid/recaptcha": "^1.0.4-alpha.1" and it seems to be working fine in the project. Phoenix |
Thank you @Risen-Solutions! @mvrlin could you take a look at the PR? Thanks! |
@andrewmurraydavid Thanks for the PR! |
@andrewmurraydavid yipp works, thank you 👌👍 |
npm i @andrewmurraydavid/recaptcha have the same error.. |
@mangrove108 can you try to pull this change down and test it out?
You can install the npm module from the folder:
@mvrlin this PR should close #84