TYP - Facturación: Agregar detalle origen obra social"#3271
TYP - Facturación: Agregar detalle origen obra social"#3271nicolasarana wants to merge 3 commits intomasterfrom
Conversation
| const { financiador, nombre, numeroAfiliado } = this.paciente.obraSocial ? this.paciente.obraSocial : this.financiadoresPaciente[0]; | ||
| let financiadorParaSeleccion; | ||
|
|
||
| if (this.financiadorActual && this.financiadorActual.nombre) { |
There was a problem hiding this comment.
Como financiadorActual es un Object y no un valor primitivo se puede mejorar ese IF con this.financiadorActual?.nombre pero en caso de que no siempre sea un objeto y pueda tomar otros valores como booleanos, string, etc ahi si es necesario tener que usar la forma en que esta implementado.
There was a problem hiding this comment.
Se realizo los cambios pedidos y otras correciones.
|
Nico, también he notado que en las clases de los HTML no he notado cambios si no mas bien muchos linteos. Si esto es correcto, es decir, no agregaste o eliminaste nada dentro de ambas clases, entonces se podría eliminar esos cambios para una próxima. |
MarianoCampetella
changed the title
AgosLizzi
left a comment
AgosLizzi
left a comment
There was a problem hiding this comment.
Nico, sugiero para diferenciar los parentesis del nombre de la obra social, se agregue una variación tipográfica en bold.
Buenas ya fue realizado el cambio. Saludos. |
MarianoCampetella
added
changes done
and removed
changes requested
Buenas, así es, el archivo por error se subió con un mal linteo, por ahora no pudimos encontrar la forma de solucionarlo. |
|
PR ok pero revisar linteo antes de deployar |
Buenas, ya debería haber quedado arreglado el linteo en el ultimo commit. Saludos. |
4 participants
Requerimiento
TYP-20
Funcionalidad desarrollada
UserStory llegó a completarse
Requiere actualizaciones en la base de datos
Requiere actualizaciones en la API
Requiere actualizaciones en andes-test-integracion