Fran Dios

Results 178 comments of Fran Dios

Hi 👋 -- some libraries are not compatible with the [worker runtime (Oxygen)](https://shopify.dev/docs/custom-storefronts/hydrogen/deployments/oxygen-runtime) that we use. Emotion is a known library that is made for Node.js and has issues with...

Hi 👋 This is a known issue that is tracked internally. In the meantime, perhaps something like this can work: https://github.com/Shopify/hydrogen/pull/1646

Hi 👋 I believe in Remix v1 you still need to add `postcss: true` to your `remix.config.js`: https://remix.run/docs/en/1.19.3/file-conventions/remix-config#postcss

Hi! It seems it was fixed in Remix https://github.com/remix-run/remix/issues/5165 but not released yet.

PRs welcome 👍 . Stuff that need to be updated (afaik): - Use `createRoot` and `hydrateRoot` instead of `create`/`hydrate` in `entry-client`. - Remove `react-ssr-prepass` if possible and rely on new...

@hi-ogawa I'm having a similar issue with the `cookie` dependency of `@remix-run/server-runtime` when running on workerd. With NPM, `ssr.optimizeDeps: {include: ['cookie']}` works well. However, with PNPM the cookie package is...

>Regarding @remix-run/server-runtime and cookie, I remember the same error and what I did is to put the entire @remix-run/server-runtime as ssr.optimizeDeps.include (probably I wasn't familiar with ... > ... syntax...

Hi Dan, thanks for bringing this up! It's indeed confusing so let me clarify a couple of things: Hydrogen is now built on top of [Remix](https://remix.run/), and it's actually the...

Thanks for the quick response! > (and admittedly silence all workerd logs, but this is probably ok if an inspector is connected) I'd consider this a feature since right now...

@nfriedly Hi 👋 would you consider adding a small script to generate the ESM version? Somewhat similar to this PR in `cookie` for the same purpose: https://github.com/jshttp/cookie/pull/154 It doesn't add...