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

[Bug?]: CSR still builds server code #1074

Closed
2 tasks done
stickyburn opened this issue Sep 30, 2023 · 1 comment
Closed
2 tasks done

[Bug?]: CSR still builds server code #1074

stickyburn opened this issue Sep 30, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@stickyburn
Copy link
Contributor

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Current behavior 😯

  • Develop a CSR app
  • Add client side library like supabase
  • Build with pnpm run build
  • SSR code is outputted, with errors due to client lib requiring node environment

Expected behavior 🤔

Should generate client side code only.

Steps to reproduce 🕹

Steps:

  1. create new project, n for SSR and barebones template
  2. add client lib
  3. build prod bundle
  4. SSR code is generated, with errors in build process

Context 🔦

Wanted to use solid-start with CSR to build a MVP. Things worked amazingly till I added supabase and started getting deployment errors. Seems like it's generating code for SSR, because a replica setup on plain solid and router works as expected.

Might be related to #978

Your environment 🌎

Arch linux
Node v18.18.0
Pnpm 8.8.0
solid-start ^0.3.6
@stickyburn stickyburn added the bug Something isn't working label Sep 30, 2023
@ryansolid
Copy link
Member

The new CSR isolation I believe should help with this. Keep in mind CSR does require a server to render index.html but we now don't import the app on the server.. only the entry server.

In setting up for SolidStart's next Beta Phase built on Nitro and Vinxi we are closing all PRs/Issues that will not be merged due to the system changing. If you feel your issue was closed by mistake. Feel free to re-open it after updating/testing against 0.4.x release. Thank you for your patience.

See #1139 for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants