Se cambia respuesta de json para obtener graficas Pie

kebblar / petstore-back

Pet Store app Backend

MIT License

0 stars 1 forks source link

Se cambia respuesta de json para obtener graficas Pie #288

Closed UALU closed 3 years ago

UALU commented 3 years ago

Se suben cambios para actualizar graficas Pie

arellano-gustavo commented 3 years ago

Que error está arreglando este Fix? El String que ahora regresas tiene forma de Json. Porqué no regresar un arreglo de objetos en vez del String en forma de json?

UALU commented 3 years ago

Buena tardes, anteriormente, se regresaba un arreglo de objetos, en esta ocasión el String tiene el formato esperado para rellenar la grafica, ya esta probado en el ambiente local, este ajuste surge por la necesidad de cambiar el tipo de grafica

arellano-gustavo commented 3 years ago

Hola, Jóvenes. Buenas tardes. Veo un problema "técnico...". Los controladores aunque pueden, no deben regresar cadenas de tipo String pensando que van a ser json. Tienen unos convertidores (formatearGraficaPaqueteria y formatearGraficaMascota) que regresan cadenas isomorfas, o sea, idénticas en estructura, aunque diferentes en el nombre de un atributo. Creo que eso se puede resolver de una mejor manera, mas canónica (con clases polimórficas) por decirlo de alguna forma. Me gustaría tener una reunión con ustedes para ver ese punto. ¿Cuándo pueden? Mil Gracias !!!!!

UALU commented 3 years ago

Claro, te contacto por Hangs

arellano-gustavo commented 3 years ago

Ok. Escribanme a la cuenta de "diplomado", por favor.

UALU commented 3 years ago

Se actualiza con ChartWrapper, se hace limpieza de codigo, lo que no eh conseguido es regresar como objeto ChartWrapper, al formatear con gson.ToJson(), regresa un String

UALU commented 3 years ago

Se completa el codigo regresando un json en lugar de un String, quedo al pendiente