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

Lpd 3472 final test #5877

Closed
wants to merge 21 commits into from
Closed

Conversation

brianikim
Copy link
Member

https://liferay.atlassian.net/browse/LPD-3472

@luca-pellizzon ,

I tested the front-end in SKU/UOM Edit view, Inventory, and also discounts.

Please let me know if I missed anything.

…cale.

# breaking

## What modules/apps/commerce/commerce-api/src/main/java/com/liferay/commerce/util/CommerceQuantityFormatter.java

modules/apps/commerce/commerce-api/src/main/java/com/liferay/commerce/util/CommerceQuantityFormatter.java has a new method parse(BigDecimal quantity, Locale locale);

## Why

this change is required to correctly parse non-US locale inputs for price and quantity.

----

# breaking

## What modules/apps/commerce/commerce-api/src/main/java/com/liferay/commerce/util/CommerceOrderItemQuantityFormatter.java

modules/apps/commerce/commerce-api/src/main/java/com/liferay/commerce/util/CommerceOrderItemQuantityFormatter.java has a new method format(CommerceOrderItem commerceOrderItem, Locale locale);

## Why

this change is required to correctly format non-US locale inputs for price and quantity.

----
# breaking

## What modules/apps/commerce/commerce-discount-api/src/main/java/com/liferay/commerce/discount/exception/CommerceDiscountMinPriceValueException.java

modules/apps/commerce/commerce-discount-api/src/main/java/com/liferay/commerce/discount/exception/CommerceDiscountMinPriceValueException.java was introduced as a new exception;

## Why

this change is required to validate for min price values.

----

# breaking

## What modules/apps/commerce/commerce-price-list-api/src/main/java/com/liferay/commerce/price/list/exception/CommercePriceListMinPriceValueException.java

modules/apps/commerce/commerce-price-list-api/src/main/java/com/liferay/commerce/price/list/exception/CommercePriceListMinPriceValueException.java was introduced as a new exception;

## Why

this change is required to validate for min price values.

----

# breaking

## What modules/apps/commerce/commerce-product-api/src/main/java/com/liferay/commerce/product/exception/CPInstanceMinPriceValueException.java

modules/apps/commerce/commerce-product-api/src/main/java/com/liferay/commerce/product/exception/CPInstanceMinPriceValueException.java was introduced as a new exception;

## Why

this change is required to validate for min price values.

----
@brianikim brianikim closed this Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant