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

[WD-17199] job apply recaptcha #1456

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

Conversation

mariuskava
Copy link
Contributor

@mariuskava mariuskava commented Dec 11, 2024

Done

Added recaptcha verification on job application submit. Recaptcha is activated on /careers pages.

A flying recaptcha logo is visible in the bottom right:
image

Error template is re-used to display failed recaptcha verification:
image

QA

  • Check out this feature branch
  • Check if GREENHOUSE_DEBUG=true (There should be a message in the logs to indicate that)
  • Run the site using the command dotrun
  • View the site locally in your web browser at: http://0.0.0.0:8002/
  • Navigate to /careers and try to apply for a job.
  • To simulate failed recaptcha set RECAPTCHA_SCORE_THRESHOLD="1.1"
  • To simulate cached page (without recaptcha) - revert templates/base_index.html and templates/careers/job-detail.html. Submit should succeed when RECAPTCHA_ENABLED="false"

@webteam-app
Copy link

@mariuskava
Copy link
Contributor Author

@anthonydillon Should we make any changes to our cookie policy popup regarding this?

Recaptcha is not explicitly mentioned but it kinda falls under the "Essential" category for access to secure areas. Also the recaptcha logo has its own links to "Privacy" and "Terms".

image

image

image

@anthonydillon
Copy link
Contributor

Does it set a cookie? Maybe we should add it too: https://ubuntu.com/legal/data-privacy#:~:text=Not%20specified-,Cookies,improve%20our%20site%20and%20services. Which is liked to from all sites regarding cookies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants