Skip to content

Conversation

engcom-Kilo
Copy link
Contributor

Description (*)

Fixed Issues (if relevant)

  1. Need to add notification message that reCAPCTHA type should be configured before enabling it  #188: Need to add notification message that reCAPCTHA type should be configured before enabling it

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 Component: Admin Component: Google reCAPTCHA Issues and Pull Requests related to reCAPTCHA should be marked with this label labels Apr 7, 2020
<group id="type_for">
<field id="recaptcha_frontend_info_heading" translate="label" sortOrder="100" showInDefault="1"
showInWebsite="1" showInStore="0">
<frontend_model>Magento\ReCaptchaAdminUi\Block\Adminhtml\System\Config\Form\Field\Notice</frontend_model>
Copy link
Contributor

Choose a reason for hiding this comment

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

Need to move this to system.xml of ReCaptchaAdminUi module.

<field id="user_login" translate="label" type="select" sortOrder="100" showInDefault="1"
<field id="recaptcha_backend_info_heading" translate="label" sortOrder="100" showInDefault="1"
showInWebsite="0" showInStore="0" >
<frontend_model>Magento\ReCaptchaAdminUi\Block\Adminhtml\System\Config\Form\Field\Notice</frontend_model>
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

@lenaorobei lenaorobei added this to the 1.0.0 milestone May 21, 2020
@lenaorobei lenaorobei changed the base branch from 1.0-develop to 1.0.0-develop May 21, 2020 19:39
@lenaorobei
Copy link
Contributor

@magento run all tests

@engcom-Delta engcom-Delta self-assigned this May 22, 2020
@engcom-Delta
Copy link

@engcom-Kilo Word "Important" in notice for Google reCAPTCHA Admin Panel is black instead of red
image

Could you take a look?

@engcom-Kilo
Copy link
Contributor Author

Could you take a look?

Updated CSS styles.

@lenaorobei
Copy link
Contributor

@magento run all tests

@engcom-Delta
Copy link

✔️ QA passed
Result:

  • Google reCAPTCHA Admin Panel

image

  • Google reCAPTCHA Storefront

image

@lenaorobei
Copy link
Contributor

@magento run all tests

@lenaorobei lenaorobei merged commit b1fbd95 into 1.0.0-develop May 27, 2020
@lenaorobei lenaorobei deleted the security-package/issues/188 branch June 10, 2020 20:55
magento-devops-reposync-svc pushed a commit that referenced this pull request Apr 9, 2025
Merge pull request #194 from magento-cia/cia-2.4.8-develop-2.4-develo…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Admin Component: Google reCAPTCHA Issues and Pull Requests related to reCAPTCHA should be marked with this label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need to add notification message that reCAPCTHA type should be configured before enabling it
3 participants