Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remix example broken #30735

Closed
2 tasks done
janat08 opened this issue Jan 22, 2022 · 7 comments
Closed
2 tasks done

Remix example broken #30735

janat08 opened this issue Jan 22, 2022 · 7 comments
Labels
bug 🐛 Something doesn't work external dependency Blocked by external dependency, we can’t do anything about it

Comments

@janat08
Copy link

janat08 commented Jan 22, 2022

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Current behavior 😯

emotion-js/emotion#2413
Running the app does nothing if using cloudflare workers variant of remix-run (generate remix-run) for cloudflare workers which uses esmodules.

Expected behavior 🤔

For the example project to work, since the whole point of it is to not have to play with bundlers and stuff.

Steps to reproduce 🕹

Steps:

  1. Create remix-run for cloudflare workers
  2. Run it and visit 8787 port from miniflare
  3. Add a component form mui to be rendered
  4. https://github.com/janat08/boughtwithslowsearchfront/commit/30d665c212f5fcbd059e0cd9bab584274112c3d0

Context 🔦

Deploy mui app through remix-run on cloudflare workers.

Your environment 🌎

System:
OS: Linux 5.13 Ubuntu 20.04.2 LTS (Focal Fossa)
Binaries:
Node: 16.13.0 - ~/.nvm/versions/node/v16.13.0/bin/node
Yarn: 1.22.17 - ~/.nvm/versions/node/v16.13.0/bin/yarn
npm: 8.1.0 - ~/.nvm/versions/node/v16.13.0/bin/npm
Browsers:
Chrome: Not Found
Firefox: Not Found
npmPackages:
@emotion/react: ^11.7.1 => 11.7.1
@emotion/styled: ^11.6.0 => 11.6.0
@mui/base: 5.0.0-alpha.65
@mui/material: ^5.3.0 => 5.3.0
@mui/private-theming: 5.3.0
@mui/styled-engine: 5.3.0
@mui/system: 5.3.0
@mui/types: 7.1.0
@mui/utils: 5.3.0
@types/react: ^17.0.24 => 17.0.38
react: ^17.0.2 => 17.0.2
react-dom: ^17.0.2 => 17.0.2
typescript: ^4.1.2 => 4.5.5

@janat08 janat08 added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Jan 22, 2022
@janat08
Copy link
Author

janat08 commented Jan 22, 2022

emotion-js/emotion#2413
The bug that causes it to break when in esmodule, and fixes.

@mnajdova
Copy link
Member

Thanks for creating the issue and linking the corresponding issue on the emotion repository. Looks like the bug and workaround are already reported there, so I am closing this issue as it is a problem in an external dependency.

@mnajdova mnajdova added bug 🐛 Something doesn't work external dependency Blocked by external dependency, we can’t do anything about it and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 24, 2022
@janat08
Copy link
Author

janat08 commented Jan 24, 2022 via email

@jgb-solutions
Copy link

Just ran into the same issue here with Cloudflare Pages with Remix and MUI. Any solutions?

@eduardosanzb
Copy link

@jgb-solutions did you solved it? I just started trying to setup this; seems to be an issue with emotion using some nodejs dependencies.

@jgb-solutions
Copy link

@eduardosanzb yeah I got it working by not using MUI on Cloudflare. I think I ended up using Vercel and fly.io to deploy the site.

@eduardosanzb
Copy link

awesome; I ended up following this comment @jgb-solutions emotion-js/emotion#2446 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work external dependency Blocked by external dependency, we can’t do anything about it
Projects
None yet
Development

No branches or pull requests

4 participants