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

disable add to until page load #20964

Conversation

sunilit42
Copy link
Contributor

Description (*)

Open a configurable product having DropDown options
When the page is being loaded, you will notice that the dropdowns has no values
Click Add To Cart button during this time. The page will be reloaded and a notice message "Open a configurable product having DropDown options" will be shown

Contribution checklist (*)

  • 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 on Travis CI are green)

@magento-cicd2
Copy link
Contributor

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ sunilaccorin
❌ Sunil Patel


Sunil Patel seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

@magento-engcom-team
Copy link
Contributor

Hi @sunilit42. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

  • @magento-engcom-team give me test instance - deploy test instance based on PR changes
  • @magento-engcom-team give me 2.3-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

@sunilit42
Copy link
Contributor Author

@orlangur i did changes, please check it again

@sidolov
Copy link
Contributor

sidolov commented Feb 4, 2019

Hi @sunilit42 , looks like you made some commits with email different than in your GitHub profile, please, add email from commits to your profile!
Thank you!

@@ -13,7 +13,8 @@ define([
$.widget('mage.productValidate', {
options: {
bindSubmit: false,
radioCheckboxClosest: '.nested'
radioCheckboxClosest: '.nested',
addToCartButtonSelector: '.action.tocart'
Copy link
Contributor

Choose a reason for hiding this comment

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

Again, CSS classes cannot be used as a JavaScript logic selectors. Please check examples in link I posted.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@orlangur magento itself using that same class into add to cart js so i am using that
https://github.com/magento/magento2/blob/2.3-develop/app/code/Magento/Catalog/view/frontend/web/js/catalog-add-to-cart.js#L23

let me know if still you want to change it

@sunilit42
Copy link
Contributor Author

sunilit42 commented Feb 4, 2019

@sidolov
what should i need to do it?
can i create new one and close to this one?

@VladimirZaets VladimirZaets self-assigned this Feb 5, 2019
@VladimirZaets
Copy link
Contributor

Hi @sunilit42. You have two options.
The first one: add email to Github that is set in git-config (git config --global user.email for look git email)
The second one: set your Github email to git config (use the same command with passing email address as the last parameter) and create new one PR

@sunilit42
Copy link
Contributor Author

@VladimirZaets

i created new pull request
#21007

@ghost
Copy link

ghost commented Feb 6, 2019

Hi @sunilit42, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@sunilit42 sunilit42 closed this Feb 6, 2019
@orlangur orlangur self-assigned this Feb 8, 2019
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.

7 participants