Skip to content

Correct typo in admin panel advanced inventory settings #2701

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

Closed
wants to merge 7 commits into from

Conversation

mbrinton01
Copy link

Advanced inventory settings in admin panel each have a label. Only one of the labels has a period at the end, which is inconsistent with the other titles: "Allow Multiple Boxes for Shipping."

This PR removes the period to make the title consistent with the others.
screen shot 2015-12-14 at 8 22 22 am

@daim2k5
Copy link
Contributor

daim2k5 commented Dec 14, 2015

@mbrinton01 please accept the CLA

@Vinai
Copy link
Contributor

Vinai commented Dec 15, 2015

Thank you for the PR! We are currently waiting for the travis builds to work again, which should happen in a few days. Once that is running and the tests are green we can continue to process your contribution.

@mbrinton01
Copy link
Author

Added a few more changes that fix a repeating typo in test suite (changing "os catalog" to "of catalog")

Correcting typo of "meesages" and grammar of message to user
@mbrinton01
Copy link
Author

Corrected another typo in user error message

@vrann
Copy link
Contributor

vrann commented Mar 22, 2017

@mbrinton01 please sync up this branch with the latest develop

@vrann
Copy link
Contributor

vrann commented Mar 22, 2017

@mbrinton01 thank you for the contribution, the issue had already been fixed in the latest develop branch.

@vrann vrann closed this Mar 22, 2017
magento-engcom-team pushed a commit that referenced this pull request Jun 15, 2018
[TSG] Backporting for 2.2 (pr30) (2.2.6)
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.

8 participants