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

Track fine grain updates to stores #21

Closed
thetarnav opened this issue Jun 30, 2022 · 0 comments · Fixed by #157
Closed

Track fine grain updates to stores #21

thetarnav opened this issue Jun 30, 2022 · 0 comments · Fixed by #157
Assignees
Labels
enhancement New feature or request priority Important issue to focus on currently solid Changes to solid are required

Comments

@thetarnav
Copy link
Owner

Currently the debugSignal, debugOwnerSignals, etc. aren't trying to handle stores specially.
In effect, debugging stores is hard or impossible.

Stores are problematic is many ways for debugging:

  • signals for properties are created only if the property is read, meaning that or I would have to read all of the keys to track them (which could affect the performance of the application in development, and provide an untrue insights, because this behavior of stores is by design) or possibly, not be able to correctly track the store state.
  • the property signals aren't added anywhere to the graph as far as I can see. So getting to them could be a challenge without changing solid code.
  • stores doesn't seem to be care about signal names, and as there is no API provided to the users to name them. Users could be stuck looking at weird, meaningless names.
@thetarnav thetarnav added the enhancement New feature or request label Jun 30, 2022
@thetarnav thetarnav added the on hold This issue is still being considered—but it's development is postponed. label Aug 4, 2022
@thetarnav thetarnav added the solid Changes to solid are required label Oct 17, 2022
@thetarnav thetarnav changed the title Logger: store support Track fine grain updates to stores Oct 17, 2022
@thetarnav thetarnav added priority Important issue to focus on currently and removed on hold This issue is still being considered—but it's development is postponed. labels Oct 21, 2022
@thetarnav thetarnav self-assigned this Oct 21, 2022
@thetarnav thetarnav mentioned this issue Oct 23, 2022
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority Important issue to focus on currently solid Changes to solid are required
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant