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

Gayatri Pai Front End Challenge #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pai-16
Copy link

@pai-16 pai-16 commented Apr 8, 2021

Thanks for the coding challenge. I've completed 4/5 tasks (excluding the reCaptcha - but comments about the process are in the code).

Thanks for the coding challenge. I've completed 4/5 tasks (excluding the reCaptcha - but comments about the process are in the code).
@pai-16 pai-16 changed the title Add files via upload Gayatri Pai Front End Challenge Apr 8, 2021
here is my understanding of how it works - obtain a site key by registering localhost.
Create a function that sends information to back end on clicking submit button.
Then verify that the response human by setting a threshold (on a scale between 0 and 1).

Choose a reason for hiding this comment

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

Did you try adding in recaptcha and it clashed or just think it will clash? Can you just modify after recaptcha the button styling?

Copy link
Author

Choose a reason for hiding this comment

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

Hi Chloe - I added in the recaptcha following this link: https://developers.google.com/recaptcha/docs/v3 where a sample submit button is created. The issue I was running into had to do with including my button style class with the given code from this page - there would be either a blank rectangle, a very small submit button, or the g-recaptcha button on top of the style attributes I created. Would you like me to try implementing recaptcha by itself?

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