AM - Se refactoriza algo de código #228
Conversation
| <ion-card-header> | ||
| <ion-avatar> | ||
| <ion-icon *ngIf="a?.link" name="desktop-outline" color="primary" class="icono-info"> | ||
| <ion-icon *ngIf="agenda?.link" name="desktop-outline" color="primary" class="icono-info"> |
There was a problem hiding this comment.
Por qué se consulta si agenda existe? Puede ser undefined?
There was a problem hiding this comment.
Tambien si lo unico que cambia si existe el link o no es el nombre, se podría hacer
<ion-icon [name]="agenda?.link ? 'desktop-outline' : 'andes-turno'" color="primary" class="icono-info"></ion-icon>
There was a problem hiding this comment.
Se agrega el cambio.
| *ngIf="b.restantesProgramados > 0 && (b.restantesMobile > 0 || a.cumpleRegla) && incluyePrestacion(b)"> | ||
| <ng-container *ngFor="let t of b.turnos"> | ||
| <ion-item *ngIf="disponible(t)"> | ||
| *ngIf="bloque.restantesProgramados > 0 && (bloque.restantesMobile > 0 || agenda.cumpleRegla) && incluyePrestacion(bloque)"> |
There was a problem hiding this comment.
se podria extraer esto en una funcion de componente para evitar tener tanta lógica en el html y hacerlo más legible
<ng-container *ngIf="debeMostrarBloque(bloque, agenda)">
There was a problem hiding this comment.
Agregada la sugerencia.
Gracias Juli, muy buenas observaciones. |
Requerimiento
https://proyectos.andes.gob.ar/browse/AM-331
Funcionalidad desarrollada
UserStory llegó a completarse
Requiere actualizaciones en la base de datos
Requiere actualizaciones en la API
Requiere actualizaciones en andes-test-integracion