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

Clean redux implementation #70

Open
r0one opened this issue Jul 27, 2019 · 5 comments
Open

Clean redux implementation #70

r0one opened this issue Jul 27, 2019 · 5 comments

Comments

@r0one
Copy link
Contributor

r0one commented Jul 27, 2019

Please help me with:

  • listing all files with dirty state management or network I/O (just edit the following list)
  • fix them by implementing interfaces with redux; then reference this issue in your PR and check the box associated with the dirty file.
@NastuzziSamy
Copy link
Contributor

I dont get it @CdRom1, what's your problem?

Redux is well implemented and quite customized, in order to auto generate actions and store in a dynamic storage. Check the code, a lot of usage examples are present.

Can you be more explicit in your demand?

@r0one
Copy link
Contributor Author

r0one commented Jul 27, 2019

  • 'src/services' :
    • 'Api.js'
    • 'Portail.js'
    • 'ActualitesUTC.js'

@NastuzziSamy
Copy link
Contributor

Ouh sry, wrong repo, redux is not used here, do you think it is usefull to implement redux ?

@r0one
Copy link
Contributor Author

r0one commented Jul 27, 2019 via email

@NastuzziSamy
Copy link
Contributor

I complelty agree 😊

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

No branches or pull requests

2 participants