Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Oscar Martín Escribano - 03942162P #18

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

ker006
Copy link

@ker006 ker006 commented Sep 8, 2024

  1. He utilizado el https://api.opendata.esett.com/EXP05/Fees ya que era el más simple y contenía datos "planos". No habría sido problema utilizar cualquier otro ya que he implementado Code First y la BD se crea al vuelo en base a la migración de Entity Framework, asique en caso de utilizar otro endpoint que contuviese un json con múltiples nodos simplemente habría que hacer la relación correcta de entidades.
  2. He implementado DDD para el diseño del API.
  3. No he implementado TDD ya que he comenzado con la funcionalidad antes que los tests dándole prioridad a la misma (gajes de trabajar con equipos pequeños con metodología agile y estar muy en contacto con las necesidades de negocio).
  4. He añadido los comentarios al final por comodidad personal, ya que solo trabajo yo sobre este fork, en caso de haber trabajado varias personas en la misma rama habría añadido los comentarios a medida que fuese haciendo cada funcionalidad.
  5. He dejado conectada la capa de Application aunque para la escritura en BBDD y lectura no la he usado. He dejado un comentario en el codigo indicando que si se fuesen a modificar los registros de Fees aplicando algún tipo de lógica de negocio, se debería implementar en esa capa al igual que las validaciones.
  6. No he usado nunca la mónadas como tal, pero tras ojear el concepto, creo que la implementación del DTO Base Response responde al uso de la mónada Result aunque no implemente el uso de binding. Al comienzo cree una clase Either para implementarlo como mónada, pero al ir a usarlo en los resultados de los servicios me di cuenta que en el propio BaseResponse ya muestro los errores o el resultado correcto (aunque no tenga una estructura L-R).
  7. Para ser sinceros, me comentaron que la prueba debería terminarse en una hora, pero me ha llevado 1 día y en lo que más he tardado es en configurar las variables de entorno del contenedor db en docker-compose.yml ya que nunca había usado docker.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants