Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

Add types to the Redux store #259

Open
mtfranchetto opened this issue Aug 18, 2022 · 0 comments
Open

Add types to the Redux store #259

mtfranchetto opened this issue Aug 18, 2022 · 0 comments
Labels
tech-debt Legacy issues or new tools adoption

Comments

@mtfranchetto
Copy link
Contributor

Description

Types are great. Following what has been done with #244 we should add types to the Redux store as well.

The types can be static (added manually) or dynamically, based on the reducers composing the store.
Given the dynamic nature of the PCM store, and all the related setState/useState calls, a static typing it's probably easier to implement, even if changes to the store will require a manual update to the types.

Stretch goal

Add typings also for setState/useState functions to have type checking for every operation on the store and avoid manual upcast/downcast.

@mtfranchetto mtfranchetto added the tech-debt Legacy issues or new tools adoption label Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tech-debt Legacy issues or new tools adoption
Projects
None yet
Development

No branches or pull requests

1 participant