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

[ECP-9098] When Terms and Condition checkbox is required Google pay and apple not validating it #2547

Open
sarfarazbheda opened this issue Mar 7, 2024 · 9 comments
Assignees
Labels
Bug report Indicates that issue has been marked as a possible bug

Comments

@sarfarazbheda
Copy link

Making your own contribution is easy, encouraged and greatly appreciated! We will put effort into reviewing and merging your PR quickly. For more info, please refer to the contribution guidelines: https://github.com/Adyen/adyen-magento2/blob/develop/CONTRIBUTING.md

Describe the bug
Activate terms and conditions check box and make it required field. and tru to place order with Google pay and apple pay, it will not validate the term and condition checkbox and order will be placed successfully without selecting it.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Admin panel and activate t&c as required field
  2. Click on on Google pay or apple pay button from the checkout page payment section
    4.it will show Google payment form directly

Expected behavior
It should validate the t&c checkbox before showing Google pay payment form.

Magento version
2.4.6.-p1

Plugin version
9.2.0

Screenshots

Desktop (please complete the following information):

  • OS: All
  • Browser [e.g. chrome, safari]
  • Version All
@sarfarazbheda sarfarazbheda added the Bug report Indicates that issue has been marked as a possible bug label Mar 7, 2024
@candemiralp candemiralp changed the title When Terms and Condition checkbox is required Google pay and apple not validating it [ECP-9098] When Terms and Condition checkbox is required Google pay and apple not validating it Apr 2, 2024
@candemiralp candemiralp self-assigned this Apr 2, 2024
@candemiralp
Copy link
Member

Hello @sarfarazbheda,

Thank you for raising this issue. We've created an implementation ticket to solve this issue.

Best Regards,
Can

@sarfarazbheda
Copy link
Author

@candemiralp

I have fixed this issue and raised PR also. If you can check and review it.

@Axel29
Copy link

Axel29 commented Jun 4, 2024

Hi @RokPopov, @candemiralp , any update on this issue please?

@Axel29
Copy link

Axel29 commented Jun 19, 2024

Hi again... Same question as two weeks ago, is there any update?

It's been months now and the bug is still present...

Thank you for your help.

@Axel29
Copy link

Axel29 commented Jun 21, 2024

@candemiralp I created a pull request (#2659) to fix the issue.

Everything seems to work with this code.

I don't know if this can be applied to this issue too, but I'll let you check.

@RokPopov
Copy link
Contributor

Hi @Axel29,

thank you for creating a PR, this is much appreciated. We have included the reviewing of the PR in the internal ticket. The issue has been prioritized accordingly internally, and we'll update this ticket once the solution has been reviewed.

Kind regards,
Rok

@sarfarazbheda
Copy link
Author

Hello,

I already created PULL request and fixed this issue, if you can check and approve it.
#2548

@Axel29
Copy link

Axel29 commented Jun 26, 2024

Hello,

I already created PULL request and fixed this issue, if you can check and approve it. #2548

Hi @sarfarazbheda , your PR didn't work for me with PayPal.
I'm not sure our bugs are the same, but @RokPopov closed my issue in order to merge it with yours.
My fix only concerns PayPal.

@candemiralp
Copy link
Member

Hello @sarfarazbheda,

Thank you for raising this issue and appreciate your efforts for creating a pull request to solve this issue.

However, the fix #2548 can not be applied to adyen-pm-method.js file without any further logic as @khushboo-singhvi mentioned here. As the requested changes were not applied, the PR has not been merged.

The reason behind is that onClick event handler is not supported by all payment method components. The scope of this modifications should be limited in order not to affect the rest of the payment methods.

I hope this explains the reason behind the delay and not merging the PR. I would like to inform you that we will handle this bug soon.

Best Regards,
Can

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug report Indicates that issue has been marked as a possible bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants