IT-Academy-BCN / ita-challenges-backend

Backend of ITA Challenges Project
MIT License
14 stars 12 forks source link

Get Solutions: visualización de soluciones aceptadas de un Challenge #468

Open jonatanvicente opened 7 months ago

jonatanvicente commented 7 months ago

Es necesario habilitar el endpoint /solution/{idChallenge}/language/{idLanguage} para que permita la visualización de varias soluciones aceptadas. Debe devolver un array con las solutions.

Véase como formato de salida https://github.com/IT-Academy-BCN/ita-challenges-backend/files/13296014/solution.challenges.out.txt

NOTA: Parece que el endpoint ya está en DEV, pero no devuelve correctamente alguno de los campos (uuid_challenge = null);

christianamor3 commented 7 months ago

Hay que revisar que las soluciones de los retos sean del mismo lenguage que el reto. Si la solucion tiene un lenguaje java (por.ej) y el reto es de PHP, no cogerá la solución.

Si añadimos varias soluciones a un reto si sale en formato array.

christianamor3 commented 7 months ago

He tenido que crear un TrimmedSolutionDto para convertir las SolutionDto completas a SolutionDto recortadas.

Inicialmente habia hecho un conversor de dto completa a recortada, pero no podia pasarle correctamente el flujo resultante al ChallengeDto, y me seguia cogiendo las completas. Entonces he tenido que pasar las SolutionDto a TrimmedSolutionDto y ahora si me las coge el ChallengeDto.

christianamor3 commented 7 months ago

De aceptar esta solucion, habría que arreglar los test. Si no, puedo volver a un commit antiguo y no hacer lo del TrimmedSolutionDto.