Skip to content

refactor(cart): group cart-discount into cart#810

Merged
CarlosCortizasCT merged 4 commits into
mainfrom
nm-group-models-cart
May 19, 2025
Merged

refactor(cart): group cart-discount into cart#810
CarlosCortizasCT merged 4 commits into
mainfrom
nm-group-models-cart

Conversation

@emmenko
Copy link
Copy Markdown
Member

@emmenko emmenko commented May 12, 2025

https://commercetools.atlassian.net/browse/FEC-187

Group cart related models into ./cart:

/cart
  /cart
  /cart-discounts

@emmenko emmenko requested review from a team as code owners May 12, 2025 14:44
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 12, 2025

🦋 Changeset detected

Latest commit: e2d8194

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 Patch

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

@emmenko
Copy link
Copy Markdown
Member Author

emmenko commented May 12, 2025

@CarlosCortizasCT Are there any other models that need to be moved into ./cart?

@CarlosCortizasCT
Copy link
Copy Markdown
Contributor

@CarlosCortizasCT Are there any other models that need to be moved into ./cart?

You can find the whole new structure in the RFC.

I don't think we planned for any other model to go to this folder.

@CarlosCortizasCT CarlosCortizasCT added the fe-chapter-rotation Tasks coming from frontend chapter work label May 12, 2025
@emmenko
Copy link
Copy Markdown
Member Author

emmenko commented May 12, 2025

Yes I'm following the RFC. The ./cart folder already contains lots of other models (which weren't listed in the RFC). That's why I'm asking 😊

@CarlosCortizasCT
Copy link
Copy Markdown
Contributor

Yes I'm following the RFC. The ./cart folder already contains lots of other models (which weren't listed in the RFC). That's why I'm asking 😊

If you take a look at the RFC you can see how the current cart folder was supposed to be a subfolder of the new parent cart folder which would also contain the cart-discount folder.
image

Let's agree on what to do here and the remaining work in our daily tomorrow.

@emmenko emmenko marked this pull request as draft May 14, 2025 11:43
@emmenko emmenko force-pushed the nm-group-models-cart branch from d0f9cf0 to 73d7e6d Compare May 16, 2025 07:33
Comment thread babel.config.js Outdated
@emmenko emmenko marked this pull request as ready for review May 16, 2025 09:21
@emmenko emmenko requested review from a team as code owners May 16, 2025 09:21
@emmenko emmenko requested review from a team and removed request for a team May 16, 2025 09:22
@CarlosCortizasCT
Copy link
Copy Markdown
Contributor

@emmenko would you be able to take a look at the typescript validation errors?

@emmenko
Copy link
Copy Markdown
Member Author

emmenko commented May 16, 2025

I looked into it but I don't know what the issue is to be honest. It only happens with the production build.

I'll continue to investigate later.

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 a lot 👍

Copy link
Copy Markdown
Contributor

@rajrdk rajrdk left a comment

Choose a reason for hiding this comment

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

@CarlosCortizasCT, should we replace paths with alias? I noticed they're used in many files.

@CarlosCortizasCT
Copy link
Copy Markdown
Contributor

@CarlosCortizasCT, should we replace paths with alias? I noticed they're used in many files.

That would be great but it's not strictly necessary.
Most of them are in sample data presets which will be moved out of this repository anyway (reference) so I wouldn't spend the time now.

@CarlosCortizasCT CarlosCortizasCT merged commit f0e33e3 into main May 19, 2025
8 checks passed
@CarlosCortizasCT CarlosCortizasCT deleted the nm-group-models-cart branch May 19, 2025 10:31
@ct-changesets ct-changesets Bot mentioned this pull request May 19, 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.

4 participants