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.

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

Move the customer search by id to the first place #16074

Merged
merged 1 commit into from Dec 5, 2019

Conversation

@levyn
Copy link
Contributor

levyn commented Oct 22, 2019

Questions Answers
Branch? develop
Description? When creating a new address from an order in the BackOffice, the address is assigned to the wrong user
Type? bug fix
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #16116
How to test? 1. Create few guest or non-guest accounts (with the same email address).
2. Start creating a new order from the Backoffice
3. Select the last account you have created
4. Create a new address for this user
Error: the address will be assigned to the first from previously created account

This change is Reviewable

@levyn levyn requested a review from PrestaShop/prestashop-core-developers as a code owner Oct 22, 2019
@prestonBot

This comment has been minimized.

Copy link
Collaborator

prestonBot commented Oct 22, 2019

Hello @levyn!

This is your first pull request on the PrestaShop project. Thank you, and welcome to this Open Source community!

@PierreRambaud

This comment has been minimized.

Copy link
Contributor

PierreRambaud commented Oct 23, 2019

Hi and thanks for your contribution.
Could you please create an issue to determine and better understand the problem with our Product team?

@PierreRambaud

This comment has been minimized.

Copy link
Contributor

PierreRambaud commented Dec 4, 2019

@levyn Any news?

@levyn

This comment has been minimized.

Copy link
Contributor Author

levyn commented Dec 4, 2019

@PierreRambaud
I'm sorry, but what are you about?
The issue #16116 was created and linked to the current issue of the.
Is there something missing here?

@PierreRambaud

This comment has been minimized.

Copy link
Contributor

PierreRambaud commented Dec 4, 2019

@levyn Didn't have answer from you, and GitHub doesn't notify us when a comment is edited, sorry :(
It's ok now :)

@sarahdib

This comment has been minimized.

Copy link

sarahdib commented Dec 5, 2019

Thank you @levyn for this contribution :)

@PierreRambaud PierreRambaud merged commit 414dfbc into PrestaShop:develop Dec 5, 2019
2 checks passed
2 checks passed
PrettyCI Code formatting
Details
Travis CI - Pull Request Build Passed
Details
@PierreRambaud

This comment has been minimized.

Copy link
Contributor

PierreRambaud commented Dec 5, 2019

Thanks @levyn

@PierreRambaud PierreRambaud added this to the 1.7.7.0 milestone Dec 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.