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

1.2 #215

Closed
wants to merge 16 commits into from
Closed

1.2 #215

wants to merge 16 commits into from

Conversation

ctippler
Copy link
Contributor

No description provided.

jan888adams and others added 16 commits November 22, 2024 10:48
…tal (#200)

* Set amount to the subtotal when the discount amount exceeds the subtotal

* Make not discounting price modifications optional

* Remove deprecation
* [WIP] Initial opensearch draft

* Apply php-cs-fixer changes

* Indexing works with opensearch

* add small cleanup

* add small cleanup

* Check productlisting with opensearch tenant

* Apply php-cs-fixer changes

* Replace elastic search everywhere

* add more cleanup

* add more cleanup

* add more cleanup

* add original sync command

* remove debug

* Apply php-cs-fixer changes

* Add deprecations and duplicate filtertypes for opensearch

* Apply php-cs-fixer changes

* Add docs for OpenSearch

* Apply php-cs-fixer changes

* Add to ignore 404 on create-index

* Apply php-cs-fixer changes

* fix: ignore 404 as we handle it differently

* fix: ignore 404 as we handle it differently

* Prepare groups for multi select

---------

Co-authored-by: mattamon <[email protected]>
Co-authored-by: lukmzig <[email protected]>
Co-authored-by: lukmzig <[email protected]>
…th a name that exists

When you create a Pricing Rule with an name that already exists the backend interface is not loading the pricing rules anymore because it inserts a row with "null" values.

The reason is because the create() call is successfully but at the update() call you get an errror because a pricing Rule with that name already exists. Solution -> wrap it into a transaction
…tal (#200)

* Set amount to the subtotal when the discount amount exceeds the subtotal

* Make not discounting price modifications optional

* Remove deprecation
* [WIP] Initial opensearch draft

* Apply php-cs-fixer changes

* Indexing works with opensearch

* add small cleanup

* add small cleanup

* Check productlisting with opensearch tenant

* Apply php-cs-fixer changes

* Replace elastic search everywhere

* add more cleanup

* add more cleanup

* add more cleanup

* add original sync command

* remove debug

* Apply php-cs-fixer changes

* Add deprecations and duplicate filtertypes for opensearch

* Apply php-cs-fixer changes

* Add docs for OpenSearch

* Apply php-cs-fixer changes

* Add to ignore 404 on create-index

* Apply php-cs-fixer changes

* fix: ignore 404 as we handle it differently

* fix: ignore 404 as we handle it differently

* Prepare groups for multi select

---------

Co-authored-by: mattamon <[email protected]>
Co-authored-by: lukmzig <[email protected]>
Co-authored-by: lukmzig <[email protected]>
…th a name that exists

When you create a Pricing Rule with an name that already exists the backend interface is not loading the pricing rules anymore because it inserts a row with "null" values.

The reason is because the create() call is successfully but at the update() call you get an errror because a pricing Rule with that name already exists. Solution -> wrap it into a transaction
…th a name that exists

When you create a Pricing Rule with an name that already exists the backend interface is not loading the pricing rules anymore because it inserts a row with "null" values.

The reason is because the create() call is successfully but at the update() call you get an errror because a pricing Rule with that name already exists. Solution -> wrap it into a transaction
@ctippler ctippler closed this Nov 22, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Nov 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants