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

General enhancements #38

Merged
merged 4 commits into from
Apr 5, 2021
Merged

General enhancements #38

merged 4 commits into from
Apr 5, 2021

Conversation

markerikson
Copy link
Contributor

  • Added .gitattributes file and hopefully fixed all line endings
  • Updated dev RTK dep to latest

From there, I rewrote the counter slice example:

  • Added an async status flag
  • Added a use of createAsyncThunk with a mock API call
  • Added a sync thunk that conditionally dispatches
  • Rewrote test file to use action creators
  • Added additional descriptive comments
  • Swapped +/- button ordering per JS template PR
  • Reformatted everything

- Added an async status flag
- Added a use of createAsyncThunk with a mock API call
- Added a sync thunk that conditionally dispatches
- Rewrote test file to use action creators
- Added additional descriptive comments
- Swapped +/- button ordering per JS template PR
- Reformatted everything
@markerikson markerikson merged commit e5ea07c into master Apr 5, 2021
@markerikson markerikson deleted the feature/general-enhancements branch April 5, 2021 02:15
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.

1 participant