Skip to content

Deprecate 'changed' and introduce new store update signal #30

@LPGhatguy

Description

@LPGhatguy

When I built the first version of Rodux, the best way to respond to changes outside of something like React/Roact wasn't very well understood.

I think that we understand now that the best way to respond to changes in the Rodux store is to keep the last value you had, and on update, compare the new store state with it. In short, you should ignore the second argument of the changed event!

This issue covers two gripes:

  • I want to remove the second value passed by the changed signal because it's a bad practice
  • I want to fix the naming of the changed signal to indicate that it's a signal (onUpdate maybe?)

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions