search

Domthegrom / 24

1 stars 0 forks source link

Possibly introducing redux? #9

Closed dawidvdh closed 6 years ago

dawidvdh commented 6 years ago

@Domthegrom I don't know how you feel about this, it is adding another library although it will reduce network calls (less airtime consumed from the user) by introducing a state cache, may be over kill I don't know whats important here as I am very new, @darrius feel free to jump in here in terms of importance.

With this we can also use redux-forms which is an incredibly effective way of working with forms vs the traditional react state way.

Domthegrom commented 6 years ago

@dawidvdh Not necessary currently and can be added later when scaling. At the moment firebase serves as our state as well as the navigation state.

At the scale of this application and considering the fact that it is technically a one page app redux could be considered redundant.

On top of that we have firebase which is where we manage most stuff because we cant handle sensitive things on the client side

dawidvdh commented 6 years ago

Okay awesome, makes sense in this context - closing for now.