- First commit to issue fork.
- Merge request !256Issue #2885044 by Juterpillar: Allow existing customers to edit their contact info → (Open) created by RoSk0
- last update
about 1 year ago 793 pass - Status changed to Needs review
about 1 year ago 8:53pm 15 May 2024 - last update
about 1 year ago 793 pass - last update
about 1 year ago 793 pass - 🇺🇸United States willardb
I apologize if this is not the best place for this, but I wanted to say
A) thank you for the patch and
B) although I recognize everything discussed here is for Commerce 8.x-2.x-dev, I am using 3.0.1 as of today and I updated the patch to handle a difference introduced in 3.x in this commit to OrderRefresh.php in 2021. Every time the order is refreshed, the email address for the authenticated user is syncd with the order email, which totally makes sense most of the time. But it doesn't make sense if you're allowing existing customers to update their contact info during checkout, so here's an quick updated patch that comments out the relevant bit in OrderRefresh and includes the rest of the changes provided in the original patch adjusted for a few other insignificant changes for anyone who finds this and is using 3.x. - 🇺🇸United States willardb
I apologize if this is not the best place for this, but I wanted to say
A) thank you for the patch and
B) although I recognize everything discussed here is for Commerce 8.x-2.x-dev, I am using 3.0.1 as of today and I updated the patch to handle a difference introduced in 3.x in this commit to OrderRefresh.php in 2021. Every time the order is refreshed, the email address for the authenticated user is syncd with the order email, which totally makes sense most of the time. But it doesn't make sense if you're allowing existing customers to update their contact info during checkout, so here's an quick updated patch ( commerce_checkout-v3.x-allow_existing-customers-to-edit-contact-info-2885044-3.patch ) that comments out the relevant bit in OrderRefresh and includes the rest of the changes provided in the original patch adjusted for a few other insignificant changes for anyone who finds this and is using 3.x. - 🇺🇸United States Greg Boggs Portland Oregon
Heck yea. Team work in the open source community.
- 🇮🇱Israel jsacksick
Instead of commenting out the part where the order email is synced, perhaps we should flag the order from the pane?
So basically, something like:
$order->setData('customer_email_overridden', TRUE);
This way we could skip the email refresh, when this data flag is set?
- Status changed to Needs work
2 days ago 8:01am 4 July 2025 - 🇮🇱Israel jsacksick
jsacksick → changed the visibility of the branch 3.x to hidden.
- Merge request !461Issue #2885044: Allow existing customers to edit their contact info. → (Merged) created by jsacksick
-
jsacksick →
committed 0a7c90c8 on 3.x
Issue #2885044 by rosk0, jsacksick, Juterpillar, willardb, greg boggs:...
-
jsacksick →
committed 0a7c90c8 on 3.x
- 🇮🇱Israel jsacksick
I'm wondering if I should only set the data flag for logged in customers. Perhaps no need to store extra data for anonymous customers... What do you think?
-
jsacksick →
committed d925fc45 on 3.x
Issue #2885044 followup: Only set the 'customer_email_overridden' flag...
-
jsacksick →
committed d925fc45 on 3.x