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

use redux for "state" #97

Open
stephenmathieson opened this issue Aug 14, 2016 · 4 comments
Open

use redux for "state" #97

stephenmathieson opened this issue Aug 14, 2016 · 4 comments
Labels

Comments

@stephenmathieson
Copy link
Contributor

we'd win a bunch of stuff by moving to redux:

  • easier onboarding for newcomers (everyone already knows redux, and if they don't there's a ton of documentation)
  • better developer experience provided by ridiculously awesome tools like redux-devtools-extension
  • hugely simplified codebase
  • much simpler automated testing
  • access to tons of plugins: https://www.npmjs.com/search?q=redux
@jwerle
Copy link
Contributor

jwerle commented Aug 15, 2016

I'm in full support of this !

@stephenmathieson
Copy link
Contributor Author

may be tricky, since we've got things other than regular objects in state :/

i'll hack on this a bit though, hopefully come up with something

@jwerle
Copy link
Contributor

jwerle commented Aug 15, 2016

Its a good thing those objects can be deleted from the state object though

On Mon, Aug 15, 2016 at 1:17 PM, Stephen Mathieson <[email protected]

wrote:

may be tricky, since we've got things other than regular objects in state
https://github.com/littlstar/axis/blob/master/state.js#L455-L458 :/

i'll hack on this a bit though, hopefully come up with something


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#97 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABZSvZR6fDb0HRwVQYaojn5AZW-nCf9tks5qgJ80gaJpZM4Jjyo7
.

## Joseph Werle
CTO - Littlstar
littlstar.com/werle https://littlstar.com/werle

// Download Littlstar on iOS https://itunes.apple.com/app/id964433932 and
Android
https://play.google.com/store/apps/details?id=com.littlstar.android&hl=en
//

[image: Inline image 1]

@stephenmathieson
Copy link
Contributor Author

hahahahaa k

@jwerle jwerle added the v1.x.x label Sep 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants