RossiniMaximo
commented
3 years ago Buenas Marcos, como va , gracias por la revisión y consejos! Ahi deployee los cambios para que la página persista. Me confirmarias si anda el localStorage ya ?
El dom, 3 oct 2021 a las 14:43, Marcos Reuquén @.***>) escribió:
Buenas, Máximo. Ante todo, creo que quedó muy bien y me encantó como personalizaste el juego. Hay dos detalles que estaría bueno verlos. El history queda vacío en el localStorage al salir de la web y el conteo del juego queda dentro del objeto user-data pero no guarda el dato del puntaje. Habría que agregarle el history para que cuando vuelvas siga tomando el puntaje en el que quedaste.
[image: image] https://user-images.githubusercontent.com/75027556/135765201-a1d578ff-3313-47ba-b008-34056aec63f8.png [image: image] https://user-images.githubusercontent.com/75027556/135765210-2c1b843a-6fca-4764-968d-9c0ae3d323b5.png
El otro tema es que cuando uno no elije ninguna mano y pasa el tiempo, el juego sigue dandote por perdida la partida, capaz ahí podrías decirle que si aún no eligió jugada y pasó el tiempo, que vuelva a la page de rules.
[image: image] https://user-images.githubusercontent.com/75027556/135765305-fe443ccf-ec45-4d17-93c5-53f0c0e43e68.png [image: image] https://user-images.githubusercontent.com/75027556/135765310-b8bd53cf-cf74-409a-9405-c35c780d58b4.png
Y sobre lo que me contabas que caía en recursividad cuando dabas click, me parece que es por los listeners. Creo que estás suscribiendo desde el componente de las manos y eso hace que se suscriba al state cada vez que el DOM agrega una mano en la page. Capaz, para evitarlo, podrías suscribir desde la page.
[image: image] https://user-images.githubusercontent.com/75027556/135765276-a6534157-3054-4895-a6b4-64f58229bc63.png
Saludos!
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/RossiniMaximo/desafio-apx/issues/3, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASD4RGNRXYS7FDOCRJ3K4ILUFCI27ANCNFSM5FH2ZM3A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
Marcosreuquen
Buenas, Máximo. Ante todo, creo que quedó muy bien y me encantó como personalizaste el juego. Hay dos detalles que estaría bueno verlos. El history queda vacío en el localStorage al salir de la web y el conteo del juego queda dentro del objeto user-data pero no guarda el dato del puntaje. Habría que agregarle el history para que cuando vuelvas siga tomando el puntaje en el que quedaste.
El otro tema es que cuando uno no elije ninguna mano y pasa el tiempo, el juego sigue dandote por perdida la partida, capaz ahí podrías decirle que si aún no eligió jugada y pasó el tiempo, que vuelva a la page de rules.
Y sobre lo que me contabas que caía en recursividad cuando dabas click, me parece que es por los listeners. Creo que estás suscribiendo desde el componente de las manos y eso hace que se suscriba al state cada vez que el DOM agrega una mano en la page. Capaz, para evitarlo, podrías suscribir desde la page.
Saludos!