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

TOP (Fix) - Eliminación de ultima regla #2537

Merged
merged 2 commits into from
Dec 9, 2021
Merged

TOP (Fix) - Eliminación de ultima regla #2537

merged 2 commits into from
Dec 9, 2021

Conversation

negro89
Copy link
Contributor

@negro89 negro89 commented Nov 29, 2021

se actualiza además el uso de componentes plex en el template

Requerimiento

https://proyectos.andes.gob.ar/browse/TOP-121

Funcionalidad desarrollada

  1. Actualiza uso de componentes plex
  2. Refactor para eliminación de organización de origen (con todas sus reglas) o todas las prestaciones origen de una organizacion

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

  • Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@negro89 negro89 requested review from a team as code owners November 29, 2021 21:33
@liquid36
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 5540
CYPRESS RUN: 4475
TEST START: 2021-11-29T21:46:20.731Z
TOTAL: 402
SUCCESS: 401
FAIL: 0
SKIPPED: 1

@liquid36 liquid36 added the test ok Los test estan ok label Nov 29, 2021
Copy link
Contributor

@liquid36 liquid36 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Cuando la regla queda sin organizaciones destino no me avisa que se va a borrar todo.

onSave($event) {
if (this.reglas?.length) {
save() {
if (this.reglas.length) {
if (this.reglaCorrecta) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Estaría bueno acomodar el código, verificarRegla debería devolver un booleano en vez de asignar una variable para su posterior uso. Ver todos los lugares donde se usa.

Suggested change
if (this.reglaCorrecta) {
if (this.verificarRegla()) {

<plex-list *ngFor="let prestacion of regla.origen.prestaciones; let i=index" size="sm" (click)="activarPrestacion(i)">
<plex-item>
<plex-grid cols="2" size="sm" justify="around" class="mt-0 pt-0">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

plex-grid no va a dentro de un plex-item. Qué problema tuviste?

Copy link
Contributor Author

@negro89 negro89 Dec 7, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La dificultad fue para alinear el contenido dentro del item (El checkbox queda forzadamente antes que el label, y este último queda alineado al principio. Ver imagen). El grid fue lo único que me resultó.
image

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Podes usar un ngProjectAs="plex-label" para acomodarlo segundo. Fuera de ese truco estaría bueno saber que opina @andrrrl @AgosLizzi

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coincido con @liquid36! Adjunto snippet para que el contenido del item quede alineado y el título de la prestación capitalizado:

<plex-item case="capitalize">
                <plex-label *ngIf="prestacion.prestacion?.term" titulo="{{ prestacion.prestacion.term }}"
                    [tituloBold]="false" subtitulo=""></plex-label>
                <plex-bool ngProjectAs="plex-label" [(ngModel)]="prestacion.auditable" name="prestacionAuditable{{i}}"
                    label="Auditable" class="mb-1">
                </plex-bool>
                <plex-button icon="delete" type="danger" size="sm" (click)="deletePrestacion(i)">
                </plex-button>
</plex-item>

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excelenteee. Corregido!
@liquid36 se agregó también un control/aviso para cuando se borraron las reglas existentes (No hay organización origen).

@liquid36 liquid36 added the changes requested Se solicitaron cambios label Nov 30, 2021
se actualiza además el uso de componentes plex en el template
@liquid36 liquid36 merged commit ddb2104 into master Dec 9, 2021
@liquid36 liquid36 deleted the TOP-121 branch December 9, 2021 15:43
liquid36 pushed a commit that referenced this pull request Dec 15, 2021
# [5.77.0](v5.76.0...v5.77.0) (2021-12-15)

### Bug Fixes

* **citas:** frecuentes duplicados ([#2547](#2547)) ([ca84c09](ca84c09))
* **top:** eliminacion de ultima regla ([#2537](#2537)) ([ddb2104](ddb2104))

### Features

* **CIT:** refactor visual header dar turno ([#2543](#2543)) ([b274c33](b274c33))
* **mapa-camas:** mostrar registro de diagnostico al ingreso ([#2529](#2529)) ([fb02b8c](fb02b8c))
* **misc:** visualizacion de las matriculas de un profesional ([#2540](#2540)) ([c0c6de1](c0c6de1))
* **tm:** refactor pantalla organizaciones ([#2528](#2528)) ([33e05e8](33e05e8))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes done test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants