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

CET-537/fix: Issue with saving address for logged in customers #60

Conversation

brtkwr
Copy link
Member

@brtkwr brtkwr commented Dec 6, 2024

When a customer is logged in, the address fails to save which is currently causing issues for some of our merchants.

Error: Call to undefined method Magento\Customer\Model\Data\Address::save() in /bitnami/magento/app/code/Two/Gateway/Controller/Payment/Confirm.php:162
Stack trace:

Screenshot 2024-12-05 at 09.16.42.png

@brtkwr brtkwr requested a review from a team as a code owner December 6, 2024 13:22
@brtkwr brtkwr force-pushed the brtkwr-two/cet-537-fix-issue-with-saving-address-for-logged-in-customers branch 2 times, most recently from 0d6f82f to 8520606 Compare December 6, 2024 13:38
When a customer is logged in, the address fails to save which is currently causing issues for some of our merchants.

```
Error: Call to undefined method Magento\Customer\Model\Data\Address::save() in /bitnami/magento/app/code/Two/Gateway/Controller/Payment/Confirm.php:162
Stack trace:
```

![Screenshot 2024-12-05 at 09.16.42.png](https://uploads.linear.app/dec93f74-b00f-44c6-a396-24c81f5fe6fd/42d4061f-0b22-4665-8b6e-406e14000b28/73b1181a-a299-47d5-90a3-1699ccf91074)
@brtkwr brtkwr force-pushed the brtkwr-two/cet-537-fix-issue-with-saving-address-for-logged-in-customers branch from 8520606 to 9b728a6 Compare December 6, 2024 13:45
Copy link
Contributor

@SalmanTwo SalmanTwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@brtkwr brtkwr force-pushed the brtkwr-two/cet-537-fix-issue-with-saving-address-for-logged-in-customers branch from e1a3ad9 to b4c6a7c Compare December 6, 2024 13:58
@brtkwr brtkwr force-pushed the brtkwr-two/cet-537-fix-issue-with-saving-address-for-logged-in-customers branch from b4c6a7c to 98bb264 Compare December 6, 2024 14:04
@brtkwr brtkwr merged commit b29e327 into main Dec 6, 2024
13 checks passed
@brtkwr brtkwr deleted the brtkwr-two/cet-537-fix-issue-with-saving-address-for-logged-in-customers branch December 6, 2024 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants