Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix select delivery_option when changing the address in OPC #8078

Merged
merged 1 commit into from Aug 28, 2017
Merged

Fix select delivery_option when changing the address in OPC #8078

merged 1 commit into from Aug 28, 2017

Conversation

hibatallahAouadni
Copy link
Contributor

Questions Answers
Branch? 1.6.1.x
Description? Keep the selected carrier by user when changing the address in OPC page
Type? bug fix
Category? FO
BC breaks? no
Deprecations? no
Fixed ticket? http://forge.prestashop.com/browse/PSCSX-8901
How to test? you need a customer account with two different adresses, and two different carriers that both delivers to those two adresses, then enable the OnePageCheckout. access to FO -> OPC, select the second carrier then change the delivery address to another one. The carrier will be still selected and won't change.

@basma-yangui-prestashop basma-yangui-prestashop added the Waiting for QA Status: action required, waiting for test feedback label Jul 11, 2017
@hibatallahAouadni
Copy link
Contributor Author

ping @maximebiloe this one will be merged in which version 1.6.1.16 or 1.6.1.17?

@marionf marionf added QA ✔️ Status: check done, code approved and removed Waiting for QA Status: action required, waiting for test feedback labels Jul 25, 2017
@hibatallahAouadni
Copy link
Contributor Author

hibatallahAouadni commented Aug 10, 2017 •
edited

@LittleBigDev could you please put the milestone 1.6.1.17, thanks in advance :)

@LittleBigDev LittleBigDev added this to the 1.6.1.17 milestone Aug 10, 2017
@maximebiloe maximebiloe merged commit 7a31797 into PrestaShop:1.6.1.x Aug 28, 2017
@maximebiloe
Copy link
Contributor

Thank you @hibatallahAouadni

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

@maximebiloe maximebiloe maximebiloe approved these changes

Assignees
No one assigned
Labels
QA ✔️ Status: check done, code approved
Projects
None yet
Milestone
1.6.1.17
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@hibatallahAouadni @maximebiloe @LittleBigDev @marionf @basma-yangui-prestashop