-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Correct spelling #20236
Correct spelling #20236
Conversation
Hi @ravi-chandra3197. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
@@ -280,7 +280,7 @@ define([ | |||
}, | |||
|
|||
/** | |||
* Counts total errors ammount accros all fields. | |||
* Counts total errors amount accros all fields. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Counts total errors amount accros all fields. | |
* Counts total errors amount across all fields. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @orlangur
I have updated File can you review it
@ravi-chandra3197 "accross" is not correct variant, please check my suggestion. After change, please squash everything into a single commit and force push. |
15e0e10
to
40e7a81
Compare
Hello @orlangur |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great @ravi-chandra3197! Thanks for being that responsive 👍
Hi @ravi-chandra3197, thank you for your contribution! |
Hi @ravi-chandra3197. Thank you for your contribution. |
Description (*)
Correct spelling
Fixed Issues (if relevant)
Manual testing scenarios (*)
Contribution checklist (*)