-
Notifications
You must be signed in to change notification settings - Fork 26
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
Collect category listing #820
Conversation
…tegory-listing # Conflicts: # composer.json # composer.lock
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test failed for updating category in playcart with Observer
Use the right branch from php-sdk for now to prevent PHAN checking being failed. Need to fix before merge
Need to fix the conflicts |
5b78327
to
d82dba6
Compare
…tegory-listing # Conflicts: # CHANGELOG.md
* @return AbstractCollection|CategoryCollection | ||
* @throws NostoException | ||
*/ | ||
public function buildExportCollection(Store $store, int $limit = 100, int $offset = 0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -41,7 +41,7 @@ | |||
"php": ">=7.4.0", | |||
"magento/framework": ">=101.0.6|~104.0", | |||
"ext-json": "*", | |||
"nosto/php-sdk": "^6.2" | |||
"nosto/php-sdk": "^7.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes category observer upsert query
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Documentation:
Checklist: