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

Redux Toolkit #7

Merged
merged 18 commits into from
Jul 20, 2022
Merged

Redux Toolkit #7

merged 18 commits into from
Jul 20, 2022

Conversation

lauslim12
Copy link
Owner

Quite a large PR:

  • Use Redux Toolkit (RTK) to practice global state management with it, to know more than just Context API.
  • Create Redux store, hooks, and migrate code from Context API to Slices.
  • Adjust components, states, and localStorage accordingly.
  • Delete unused code.
  • Create additional typing for media queries, and fix visual bug in some laptop screens.
  • Refactor Drinks component.
  • Debounce store subscription to prevent lag.
  • Write test utilities, made sure it integrated well with react-i18next, styled-components, and @reduxjs/toolkit.
  • Write more tests, made sure the code coverage was 100%, and fix minor bugs.
  • Update documentation.

@lauslim12 lauslim12 merged commit 7b0cc66 into main Jul 20, 2022
@lauslim12 lauslim12 deleted the redux-toolkit branch July 20, 2022 17:35
@lauslim12 lauslim12 mentioned this pull request Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant