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

Fix the customer threads status change #6654

Merged
merged 1 commit into from Oct 11, 2016

Conversation

Projects
None yet
2 participants
@antoin-m
Contributor

antoin-m commented Oct 11, 2016

Questions Answers
Branch? develop
Description? the thread's status should get updated (on the view) when you change it
Type? bug fix
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket?
How to test?
  • FO: Send a message via the "contact us" form
  • BO: Go to the customer service and click on the message
  • BO: Change the message status -> the list of available status for this message should update
@aleeks

This comment has been minimized.

Show comment
Hide comment
@aleeks

aleeks Oct 11, 2016

Contributor
  • Why did't you apply the same on the update customer ? L:315 ?
  • Can you put your Db::getInstance()->execute( on a variable to test if it's ok in order to make the change only if it's ok ??
Contributor

aleeks commented Oct 11, 2016

  • Why did't you apply the same on the update customer ? L:315 ?
  • Can you put your Db::getInstance()->execute( on a variable to test if it's ok in order to make the change only if it's ok ??
@aleeks

aleeks approved these changes Oct 11, 2016

Ok with your changes!

@aleeks aleeks merged commit 7f7b5a5 into PrestaShop:develop Oct 11, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@aleeks

This comment has been minimized.

Show comment
Hide comment
@aleeks

aleeks Oct 11, 2016

Contributor

Thank you!!

Contributor

aleeks commented Oct 11, 2016

Thank you!!

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