-
Notifications
You must be signed in to change notification settings - Fork 5
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
CIT - Ventanilla: No dar turno en agenda dinamica #2946
Conversation
ce39b5a
to
c00f8c6
Compare
c00f8c6
to
8b3f88c
Compare
} | ||
} | ||
|
||
agendaDinamicaFutura(dia) { | ||
return (dia.dinamicasIndefinidas > 0 && moment(dia.fecha).format('DD/MM/YYYY') > moment().format('DD/MM/YYYY')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ya que utilizamos moment, las comparaciones de las fechas no son a través de los símbolos <, > o = sino con isBefore, isAfter, etc. En la siguiente documentación hay varios ejemplos para trabajar sobre las comparaciones: https://momentjs.com/docs/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hecho
@@ -1320,6 +1320,18 @@ export class DarTurnosComponent implements OnInit { | |||
this.showSobreturno = !this.showSobreturno; | |||
} | |||
|
|||
agendaDeHoy() { | |||
return (moment(this.agenda.horaInicio).format('DD/MM/YYYY') === moment().format('DD/MM/YYYY')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar al comentario anterior!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hecho
Espero a la realización de los cambios propuestos por campe para finalizar review. Dejo corriendo tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Se creo tarea para analizar las mejoras visuales UIUX-374
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Juan este pr cumple con las especificaciones iniciales. Lo que menciona @AgosLizzi sobre una nueva sugerencia: https://proyectos.andes.gob.ar/browse/UIUX-374, será parte de otra tarea. Tener presente la actualización y testeo posterior.
Requerimiento
https://proyectos.andes.gob.ar/browse/CIT-249
Funcionalidad desarrollada
UserStory llegó a completarse
Requiere actualizaciones en la base de datos
Requiere actualizaciones en la API
Requiere actualizaciones en andes-test-integracion