Guilherme Carreiro

Results 83 comments of Guilherme Carreiro

👋 Hey @stijns96, I don't think that would be possible because the `sections['header-group']` syntax implies the presence of a `header-group` property on `sections` that is an array. I will close...

👋 Hey everyone, Thanks a lot for reporting this issue! This shouldn’t be happening in the latest version of Shopify CLI, but if you're still facing it, could you please...

👋 Hey folks, This issue has been resolved and the fix will be included in the next minor release. If you'd like to access the fix immediately, you can install...

I've created [this PR](https://github.com/Shopify/theme-tools/pull/738) for future reference so we can have a [diff](https://github.com/Shopify/theme-tools/pull/738/files) of the changes between the prototype and this version.

👋 Hey @charlespwd @frandiox, this PR is ready for review now :)

Thank you for the review, @frandiox! I've applied the [suggestions](https://github.com/Shopify/theme-tools/pull/669#discussion_r1935508133) :) also, I've created [this](https://github.com/Shopify/theme-tools/issues/744) to handle scenario 1, slightly updated the prompt to prevent the scenario 2 (I couldn't...

@shopify/app-inner-loop, Could you please take a look at this PR? It seems like the merge is blocked by that requirement.

👋 Hey @nboliver-ventureweb, Could you please confirm if you're using the "Shopify Translate & Adapt" app to power your translations? If that's the case, that app has the limitation that...

👋 Hey everyone, I've shared this issue internally with the team that maintains the "Shopify Translate & Adapt" app to ensure they are aware of this limitation. I also highly...

👋 Hey @wiebekaai, Thank you for reporting this! The https://github.com/Shopify/cli/pull/4570 fix also covers the use case reported here, as it targets the /cart/... prefix (and it has worked locally for...