RossiniMaximo
commented
2 years ago Buenas Marcos!! Si aún no esta al 100 , igual me re habia olvidado de la colleción de postman gracias x hacerme acordar. Gracias por los tips para acortar y emprolijar un poco, y en cuanto al storage , es una lucha que vengo peleando hace dias jajajaja , Saludos Marcos , ahora voy a ver si puedo darle la vuelta a la tortilla
El mar, 28 dic 2021 a las 8:27, Marcos Reuquén @.***>) escribió:
Buenas, Máximo!
Estuve probando y me salen algunos errores. Cuando ingreso con una "cuenta existente", no me verifica el usuario hasta el momento de jugar, y en ese momento me da como 503 Service Unavailable [image: image] https://user-images.githubusercontent.com/75027556/147560623-2805ff6c-a7f1-4d0e-9727-97b661833001.png
El parsed data arranca como null lo que hace que al hacer algunos requests a la api, envíe los datos como null. [image: image] https://user-images.githubusercontent.com/75027556/147560731-e381f6ca-93f2-4ff8-a8bf-0a83f77bf506.png Podrías agregarle una validación en el init o tal vez un const localdata = sessionStorage.getItem("data") || {userName: ""...}; [image: image] https://user-images.githubusercontent.com/75027556/147560834-b28d63d3-5259-41b6-b976-a66afc6636a9.png
Podrías emprolijar un poco el código para evitar repeticiones como la siguiente, con hacer const {ready, move, win, score,playAgain} = req.body;
https://github.com/RossiniMaximo/dwf-m6-desafio/blob/8cdef099407394373a47199554924f08be902806/server/index.ts#L125-L129 También en el state, expresiones declarativas como:
https://github.com/RossiniMaximo/dwf-m6-desafio/blob/8cdef099407394373a47199554924f08be902806/client/state.ts#L314-L320 Podrías ver de acortarlas con "copiarlo" idéntico. Me refiero a algo tipo: cs.player = player con eso te ahorrarías varias líneas declarativas y te asegurarías de tener lo mismo en el servidor y en el cliente.
Por último, faltaría en el readme la collection de Postman, marco el desafío como Necesita ajustes para que lo reenvíes con eso y de paso si podés arreglarle lo que hay en el #2 https://github.com/RossiniMaximo/dwf-m6-desafio/issues/2 y lo que te comenté.
Saludos!!
— Reply to this email directly, view it on GitHub https://github.com/RossiniMaximo/dwf-m6-desafio/issues/3, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASD4RGM3LNPEFR3VC2EOOFDUTGNKPANCNFSM5K37FANA . 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.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
Marcosreuquen
Buenas, Máximo!
Estuve probando y me salen algunos errores. Cuando ingreso con una "cuenta existente", no me verifica el usuario hasta el momento de jugar, y en ese momento me da como 503 Service Unavailable
El parsed data arranca como null lo que hace que al hacer algunos requests a la api, envíe los datos como null.
Podrías agregarle una validación en el init o tal vez un 
const localdata = sessionStorage.getItem("data") || {userName: ""...};Podrías emprolijar un poco el código para evitar repeticiones como la siguiente, con hacer
const {ready, move, win, score,playAgain} = req.body;https://github.com/RossiniMaximo/dwf-m6-desafio/blob/8cdef099407394373a47199554924f08be902806/server/index.ts#L125-L129 También en el state, expresiones declarativas como: https://github.com/RossiniMaximo/dwf-m6-desafio/blob/8cdef099407394373a47199554924f08be902806/client/state.ts#L314-L320 Podrías ver de acortarlas con "copiarlo" idéntico. Me refiero a algo tipo:cs.player = playercon eso te ahorrarías varias líneas declarativas y te asegurarías de tener lo mismo en el servidor y en el cliente.Por último, faltaría en el readme la collection de Postman, marco el desafío como Necesita ajustes para que lo reenvíes con eso y de paso si podés arreglarle lo que hay en el #2 y lo que te comenté.
Saludos!!