PracticaDS / pdes-tp-3pines

Revolución Industrial by pdes-tp-3pines
https://practicads.github.io/pdes-tp-3pines/
0 stars 1 forks source link

#20 - Mover maquina #45

Closed GianFF closed 5 years ago

GianFF commented 5 years ago

DEPENDE DEL PR ANTERIOR

Hay que agregarle tests. Si miran el código el action creator seleccionarCelda quedó bastante complejo, tiene un montón de lógica sobre si dispatchear o no INICIAR_MOVER o FINALIZAR_MOVER dependiendo de si la acción seleccionada es o no Rotar.

Quizás estaría bueno aplanar el store (eliminando los reducers que hay y dejando uno sólo) y agregar un reducer dentro del reducer general, donde el general es el que tiene la lista de celdas y el de adentro sería uno que sabe como manejar una celda en particular, quizás esto nos traería complicaciones también... no lo sé.

Toolbox-terminado