Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update checkout rebuilding #107

Merged
merged 9 commits into from
May 25, 2021
Merged

Update checkout rebuilding #107

merged 9 commits into from
May 25, 2021

Conversation

remstone7
Copy link
Contributor

No description provided.

Copy link
Contributor

@smoucka smoucka left a comment

Choose a reason for hiding this comment

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

Should we just bump the version in this pr as well?

@remstone7 remstone7 requested a review from smoucka May 18, 2021 19:49
CHANGELOG.md Outdated
@@ -6,9 +6,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### [Unreleased]

### [2.2.0] - 2021-08-18
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this should be 2021-05-18


use Magento\Quote\Api\CartRepositoryInterface;
use Magento\Quote\Api\Data\CartExtensionFactory;
use Magento\Quote\Model\QuoteIdToMaskedQuoteId;
Copy link
Contributor

Choose a reason for hiding this comment

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

what happens to customers on version pre 2.3? If you look at https://github.com/magento/magento2/tree/2.2/app/code/Magento/Quote/Model it does not exist.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we'll probably have to add this as a dependency

@remstone7 remstone7 merged commit 016114c into master May 25, 2021
@remstone7 remstone7 deleted the 202105_masked_checkouts branch May 25, 2021 13:45
@webdev2k21
Copy link

Hi,

On Magento 2.3.3 when we apply coupon code getting the error as below.

{"messages":{"error":[{"code":500,"message":"Fatal Error: 'Uncaught TypeError: Argument 2 passed to Klaviyo\Reclaim\Plugin\Api\CartSearchRepository::afterGetList() must be an instance of Magento\Quote\Model\CartSearchResults, instance of Magento\Framework\Api\SearchResults given, defined in /var/www/html/magento2/app/code/Klaviyo/Reclaim/Plugin/Api/CartSearchRepository.php:49

@remstone7
Copy link
Contributor Author

Thanks @webdev2k21 we will look into this

@klaviyojad
Copy link
Contributor

@webdev2k21 it appears to be a problem where your version does not have it like the 2.3 version magento has here: https://github.com/magento/magento2/blob/2.3/app/code/Magento/Quote/Model/CartSearchResults.php
I know they must have updated it after the fact because I can see 2.3.3 does not have it https://github.com/magento/magento2/tree/2.3.3/app/code/Magento/Quote/Model

I wonder if there is way to be still on 2.3 and get the latest updates for module-quote ?

public function getMaskedIdFromQuoteId($quoteId)
{
try {
$quoteId = $this->quoteIdMask->execute($quoteId);
Copy link
Contributor

@ihor-sviziev ihor-sviziev Jun 23, 2021

Choose a reason for hiding this comment

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

@remstone7 @klaviyojad
FYI this change causing ~2x slow down (and sometimes fatal errors with memory limit exceeded) on the quotes list retrieval request, but that's an issue in the Magneto core.
More details here: magento/magento2#33324

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants