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

Added redux for state management #16

Merged
merged 9 commits into from
Nov 29, 2019
Merged

Added redux for state management #16

merged 9 commits into from
Nov 29, 2019

Conversation

muhsin-k
Copy link
Member

@muhsin-k muhsin-k commented Nov 29, 2019

Fixes #15

  • Added redux, react-redux, redux-thunk, redux-persist, redux-logger & async-storage dependencies.
  • Changed the structure of the screen folder.
  • Added some android build files to .gitignore file.
  • Added action and reducer for locale
  • Added store in the root of the app

import {SET_LOCALE} from '../constants/actions';

export const setLocale = value => dispatch => {
dispatch({type: SET_LOCALE, payload: value});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check if you have setup eslinter/prettier correctly.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pranavrajs Eslint/Prettier is working. I think it is good If there is a line separation between import statements and other codes. This is just a suggestion, please ignore if you don't agree or we can customize the Eslint rule padding-line-between-statements.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pranavrajs Please review now

@pranavrajs pranavrajs merged commit 5983156 into master Nov 29, 2019
@pranavrajs pranavrajs deleted the 15-store-setup branch November 29, 2019 08:07
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.

Add redux for state management
2 participants