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

Add new url to categorires and coupons #52

Open
wants to merge 4 commits into
base: add-new-url-to-categorires-and-coupons
Choose a base branch
from

Conversation

mgonzaln
Copy link

Información del Candidato

Nombre: Mauricio Esteban González Núñez

Checklist del Desafío

Has una verificación final de esta lista para asegurar que no estes olvidando nada :)
No tiene problemas si necesitas hacer algún arreglo y atualizar este Pull Request después de enviarlo.

Checklist

Coloque una x en las cajas que aplican.

  • [ x] El código compila
  • [ x] Todas las pruebas se ejecutan exitosamente
  • [ x] Su proyecto está bien documentado (cómo ejecutar, pruebas ...)

Comentarios

Respecto a solución basada en caché para optimizar tiempo de respuesta, como decía en otra descripción, es configurable,e inicialmente se seteó en 1dia en memoria caché mientras se ejecuta la aplicación, al volver a ejecutar la caché se elimina.

Para la complejidad de de la base de datos, si la respuesta del servicio varía poco, bastaría implementar una replica en caché de la respuesta con alguna base de datos tipo Redis y abstraerse de la capa datos utilizando los objetos de respuesta de la api.

mauricio.gonzalezn@cajalosandes.cl added 4 commits November 17, 2021 13:04
…ados en desafio

http://localhost:8080/mobile-api/categoriasTop5
http://localhost:8080/mobile-api/categoriasNoTop5
http://localhost:8080/mobile-api/cuponesVigentes

mas info en README.MD
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.

None yet

1 participant