solid-start icon indicating copy to clipboard operation
solid-start copied to clipboard

Remove map and createStore from mount.ts

Open thetarnav opened this issue 1 year ago • 3 comments

I don't know if I'm missing something or is this a leftover, but the weak map is only being written to and the props object doesn't get changed, so it doesn't need to be a store. I don't like when stores are added to the bundle for no reason.

thetarnav avatar Oct 08 '24 18:10 thetarnav

⚠️ No Changeset found

Latest commit: 1f5b48de5b68c321ddcbb1f946edefe7a4a97dca

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.

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

💥 An error occurred when fetching the changed packages and changesets in this PR
Some errors occurred when validating the changesets config:
The package "solid-start-docs" depends on the ignored package "solid-start-mdx", but "solid-start-docs" is not being ignored. Please add "solid-start-docs" to the `ignore` option.

changeset-bot[bot] avatar Oct 08 '24 18:10 changeset-bot[bot]

It's because the implementation present here is only half of the implementation. The Store was there when we had islands routing mode which could write to the props.

ryansolid avatar Oct 09 '24 20:10 ryansolid

@thetarnav are we ok to close this PR? Or are you currently able to find it in a client bundle?

atilafassina avatar Jan 14 '25 16:01 atilafassina

I'm fine I just noticed this during my unfinished hackathon project and decided to make a pr If the fix is incorrect feel free to close this

thetarnav avatar Jan 14 '25 17:01 thetarnav