Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Conversation

ajithkumar-maragathavel
Copy link
Contributor

@ajithkumar-maragathavel ajithkumar-maragathavel commented May 10, 2021

Copy link
Contributor

@BarnyShergold BarnyShergold left a comment

Choose a reason for hiding this comment

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

Great addition - just some grammar and language corrections.

@ajithkumar-maragathavel
Copy link
Contributor Author

@BarnyShergold Thanks for the valuable suggestions. I have updated the PR. please review the changes.

Copy link
Contributor

@BarnyShergold BarnyShergold left a comment

Choose a reason for hiding this comment

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

one small change suggested, so I will approve and you can add the change - will be checked by the second reviewer


In the above configuration, the `catalog_rule_form` is a UI component form. It is defined in the `app/code/Magento/CatalogRule/view/adminhtml/ui_component/catalog_rule_form.xml` file.

The `Magento\CatalogRule\Model\Rule\Condition\Combine` and the `Magento\CatalogRule\Model\Rule\Condition\Product` classes are important for the listed conditions and validations of the specified conditions.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are these important?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dobooth Both these classes are responsible for the default conditions available in the catalog rule and defines the logic for the validations of those conditions.
That's why I termed it as important

Copy link
Contributor

Choose a reason for hiding this comment

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

I rewrote it. Please confirm if it is correct.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dobooth Yep, It is correct.

@dobooth
Copy link
Contributor

dobooth commented Jun 2, 2021

@magento import pr to magento-devdocs/devdocs

@magento-engcom-team
Copy link

@dobooth the pull request successfully imported.

@magento-cicd2 magento-cicd2 merged commit e2d37ae into magento:master Jun 4, 2021
@ghost
Copy link

ghost commented Jun 4, 2021

Hi @ajithkumar-maragathavel, 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.

@dshevtsov dshevtsov added the 2.4.x Magento 2.4.x related changes label Jun 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.4.x Magento 2.4.x related changes New Topic A major update published as an entirely new document Partner: Ziffity partners-contribution PR created by Magento partner Progress: done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add information on how to extend catalog price rule conditions
6 participants