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

BO | Stock: Edit quantity placeholder #10199

Merged
merged 4 commits into from Nov 16, 2018

Conversation

Projects
None yet
7 participants
@CaptainYouz
Contributor

CaptainYouz commented Aug 29, 2018

Questions Answers
Branch? develop
Description? This PR fix in BO > Stock Page the UX problem of the quantity placeholder.
Type? improvement
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? #9753
How to test? Change the quantity value manually, with the increase/decrease button and save it to make sure that changes are saved.

This change is Reviewable

@CaptainYouz CaptainYouz changed the title from BO | Stock: Edit quantity placeholder - #9753 to BO | Stock: Edit quantity placeholder Aug 29, 2018

@PrestaShop PrestaShop deleted a comment from prestonBot Sep 26, 2018

@PierreRambaud

This comment has been minimized.

Contributor

PierreRambaud commented Nov 15, 2018

Merge when tests are ok :)

@Quetzacoalt91 Quetzacoalt91 merged commit 50a4cf7 into PrestaShop:develop Nov 16, 2018

2 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Quetzacoalt91

This comment has been minimized.

Member

Quetzacoalt91 commented Nov 16, 2018

Thank you @CaptainYouz

@PierreRambaud PierreRambaud added this to the 1.7.6.0 milestone Nov 22, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment