search

EzequielVilla / dwf-m6-desafio

0 stars 0 forks source link

Review #1

Open Marcosreuquen opened 3 years ago

Marcosreuquen commented 3 years ago

Buenas, Eze! Está excelente!! Sobre el state como sugerencia podrías darle a la rtdb un espacio aparte en el state para ahorrarte un par de lineas de código, por ejemplo {rtdb: {playerOne: "la data", playerTwo: "la otra data"}} entonces cuando te suscribís a ella le decis que cambie lo que hay en state.data.rtdb. Y sobre la api, tal vez las interacciones con la rtdb podrías resumirlas a uno o dos endpoints con PATCH o PUT en vez de POST donde le pases el jugador y un objeto tipo {propiedad: "nuevo valor"} y con eso le hacés un update. Pero es sólo a modo de sugerencia para que te ahorres líneas. Felicitaciones!

EzequielVilla commented 3 years ago

Me sirve un montón lo segundo porque es verdad, me cebé tirando POST en lugar de PATCH para hacer el update. Con respecto a lo primero no entendí muy bien que quisiste decir. Entiendo que mi init inicializa un montón de cosas y no supe muy bien como hacer más simple eso.

EzequielVilla commented 3 years ago

Ahhhhhhh ya entendí. Decis de pasarlo como objeto?

Marcosreuquen commented 3 years ago

Claaro. Podés separa el data en dos tipo {data:{ userData: {}, rtdb: {} } } y lo que esté en userData lo podés guardar en el local storage y recuperarlo cuando quieras desde el init, inclusive los id's y lo del rtdb aparte y se actaliza solo. Más que nada porque vi que tenías que guardar todo en variables y después asignarlo a propiedades del state.