Skip to content

Conversation

@Jade-GG
Copy link
Collaborator

@Jade-GG Jade-GG commented Sep 29, 2025

We already did this in the core, but not in here.

This also works around a Magento 2.4.8-p2 bug (we think, at least) where checkout agreements were grabbed from the customer's registration store rather than the active store.

@Jade-GG Jade-GG requested a review from royduin September 29, 2025 11:02
@Jade-GG
Copy link
Collaborator Author

Jade-GG commented Sep 29, 2025

This will require a PR for 3.x and 4.x as well. I will do this soon™️.

@royduin
Copy link
Member

royduin commented Sep 30, 2025

We switch back to GraphQL in the core, see: rapidez/core#612

@Jade-GG
Copy link
Collaborator Author

Jade-GG commented Oct 1, 2025

We switch back to GraphQL in the core, see: rapidez/core#612

If we're sticking with this, we're going to have to fix the magento bug that this workaround is for. Right now multiple stores on the same website will sometimes (depending on customer registration store) give the wrong agreements.

@Jade-GG Jade-GG marked this pull request as draft December 17, 2025 10: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.

3 participants