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

fix(medusa): keep country_code when partially updating cart addresses #2683

Merged
merged 8 commits into from
Dec 12, 2022

Conversation

kasperkristensen
Copy link
Contributor

What

  • Fixes a bug where country_code was dropped when partially updating a shipping or billing address on a cart.

@kasperkristensen kasperkristensen requested a review from a team as a code owner November 28, 2022 15:58
@changeset-bot
Copy link

changeset-bot bot commented Nov 28, 2022

🦋 Changeset detected

Latest commit: 4ae289c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@medusajs/medusa Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@kasperkristensen kasperkristensen changed the title fix: keep country_code when partially updating cart addresses fix(medusa): keep country_code when partially updating cart addresses Nov 28, 2022
@kasperkristensen kasperkristensen marked this pull request as draft November 28, 2022 19:13
@olivermrbl
Copy link
Contributor

@kasperkristensen Is this ready for review?

@kasperkristensen
Copy link
Contributor Author

@olivermrbl No it requires a bit more, the "fix" we came up with the other days, solves the issue that was reported, but it breaks a bunch of integration tests.

@kasperkristensen kasperkristensen marked this pull request as ready for review December 9, 2022 12:22
@kasperkristensen
Copy link
Contributor Author

@olivermrbl Ready for review

Copy link
Contributor

@pKorsholm pKorsholm left a comment

Choose a reason for hiding this comment

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

LGTM! 😄

Copy link
Contributor

@olivermrbl olivermrbl left a comment

Choose a reason for hiding this comment

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

Great one!

Should be good to go after removing the unrelated product import file 👍

Copy link
Contributor

@olivermrbl olivermrbl left a comment

Choose a reason for hiding this comment

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

LGTM

@kodiakhq kodiakhq bot merged commit 33aa3ed into develop Dec 12, 2022
@kodiakhq kodiakhq bot deleted the fix/retain-country-code-on-update branch December 12, 2022 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants