Skip to content

FEC-241 : Migrate DiscountOnTotalPrice & DiscountedTotalPricePortion to use new model#815

Merged
rajrdk merged 8 commits into
mainfrom
fec-241-migrate-discount-on-total-price-and-discounted-total-price-portion
May 14, 2025
Merged

FEC-241 : Migrate DiscountOnTotalPrice & DiscountedTotalPricePortion to use new model#815
rajrdk merged 8 commits into
mainfrom
fec-241-migrate-discount-on-total-price-and-discounted-total-price-portion

Conversation

@rajrdk
Copy link
Copy Markdown
Contributor

@rajrdk rajrdk commented May 13, 2025

@rajrdk rajrdk requested a review from a team May 13, 2025 14:27
@rajrdk rajrdk requested a review from a team as a code owner May 13, 2025 14:27
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 13, 2025

🦋 Changeset detected

Latest commit: 539bc24

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@commercetools/composable-commerce-test-data Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@rajrdk rajrdk marked this pull request as draft May 13, 2025 14:28
@rajrdk rajrdk self-assigned this May 13, 2025
@rajrdk rajrdk added the fe-chapter-rotation Tasks coming from frontend chapter work label May 13, 2025
@rajrdk rajrdk marked this pull request as ready for review May 13, 2025 14:42
Copy link
Copy Markdown
Contributor

@CarlosCortizasCT CarlosCortizasCT left a comment

Choose a reason for hiding this comment

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

Just a couple of suggestions. Otherwise looks very good 👍

Comment thread standalone/src/models/cart/discount-on-total-price/fields-config.ts Outdated
Comment thread standalone/src/models/cart/discount-on-total-price/builders.spec.ts
@rajrdk rajrdk requested review from a team and CarlosCortizasCT May 13, 2025 18:07
Comment thread standalone/src/models/cart/discount-on-total-price/builders.spec.ts
@rajrdk rajrdk requested a review from CarlosCortizasCT May 14, 2025 08:37
Copy link
Copy Markdown
Contributor

@CarlosCortizasCT CarlosCortizasCT left a comment

Choose a reason for hiding this comment

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

I added this small commit (f3feaca) to make sure we export the types for the DiscountOnTotalPrice model and we do it the same way for that and the other new one.

Comment thread .changeset/twelve-mangos-move.md Outdated
@rajrdk rajrdk requested a review from CarlosCortizasCT May 14, 2025 12:49
Copy link
Copy Markdown
Contributor

@CarlosCortizasCT CarlosCortizasCT left a comment

Choose a reason for hiding this comment

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

Thanks! 👍

@rajrdk rajrdk merged commit 79f4d99 into main May 14, 2025
8 checks passed
@rajrdk rajrdk deleted the fec-241-migrate-discount-on-total-price-and-discounted-total-price-portion branch May 14, 2025 13:10
@ct-changesets ct-changesets Bot mentioned this pull request May 14, 2025
@rajrdk rajrdk mentioned this pull request Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fe-chapter-rotation Tasks coming from frontend chapter work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants