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

Convert base redux to TS #276

Merged
merged 2 commits into from
Nov 9, 2021
Merged

Conversation

mattrunyon
Copy link
Collaborator

Closes #274

A few things are marked as unknown and there are almost definitely some extra props missing from the RootState, but nothing that is consumed by TS. Once other things get converted to TS then the extra props can be added and corrected as needed.

@mattrunyon mattrunyon added the enhancement New feature or request label Nov 4, 2021
@mattrunyon mattrunyon requested a review from mofojed November 4, 2021 20:55
@mattrunyon mattrunyon self-assigned this Nov 4, 2021
packages/redux/src/store.ts Outdated Show resolved Hide resolved
@mattrunyon mattrunyon merged commit d0c0341 into deephaven:main Nov 9, 2021
@mattrunyon mattrunyon deleted the mattrunyon_274 branch January 15, 2022 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Convert base redux package to typescript
2 participants