search

commercetools / merchant-center-application-kit

Tools and components for developing Merchant Center Customizations 🛠
https://docs.commercetools.com/merchant-center-customizations
MIT License
66 stars 27 forks source link

refactor(application-shell): testing some changes in the application … #3527

Closed CarlosCortizasCT closed 2 months ago

CarlosCortizasCT commented 2 months ago

…context

Summary

This is just a test PR to debug some CI errors

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
merchant-center-application-kit-components-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2024 8:22am
changeset-bot[bot] commented 2 months ago

⚠️ No Changeset found

Latest commit: 73299faf43a4a905f39af044a8ed51d97d21c647

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

CarlosCortizasCT commented 2 months ago

Thanks for debugging.

If I read the PR checks right the fact of adding the field to the query causes the error to appear?

If so I can debug patching app-kit locally also maybe. I already tried a GraphQL query with the field using CURL which works.

If I read the PR checks right the fact of adding the field to the query causes the error to appear?

That's what I'm trying to find out.

I've just added the field back to the query and the tests are still green so it seems it is not the root cause.

Next step is to check the generated code.