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

security-package/issues/170: Renamed error message. #174

Closed
wants to merge 2 commits into from

Conversation

engcom-Kilo
Copy link
Contributor

Re-named error message in Resolver to - reCapctha is required.
Also corrected Integration Tests.

Description (*)

Fixed Issues (if relevant)

  1. Need to improve error message for Admin Panel if user did not check reCaptcha #170: Need to improve error message for Admin Panel if user did not check reCaptcha

Manual testing scenarios (*)

  1. ...
  2. ...

Questions or comments

Contribution checklist (*)

  • Author has signed the Adobe CLA
  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds are green)

@engcom-Kilo engcom-Kilo added the Component: Google reCAPTCHA Issues and Pull Requests related to reCAPTCHA should be marked with this label label Apr 1, 2020
@engcom-Delta engcom-Delta self-assigned this Apr 2, 2020
@engcom-Delta
Copy link

✔️ QA passed
Message with fix
image

@engcom-Foxtrot engcom-Foxtrot force-pushed the security-package/issues/170 branch from 5d21a8b to d9fe7a3 Compare April 3, 2020 08:58
@engcom-Foxtrot
Copy link
Contributor

A related issue is no longer valid due to #175.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Google reCAPTCHA Issues and Pull Requests related to reCAPTCHA should be marked with this label Progress: review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants