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.

Sign up for GitHub

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

Jump to bottom

CIT - Implementa selector de financiadores #2854

Merged
merged 1 commit into from
Dec 22, 2023
Merged

CIT - Implementa selector de financiadores #2854

merged 1 commit into from
Dec 22, 2023

Conversation

ma7payne
Copy link
Contributor

@ma7payne ma7payne commented Apr 24, 2023 •
edited by negro89
Loading

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-214

Funcionalidad desarrollada

  1. Implementación de un componente reutilizable para las obras sociales o prepagas del paciente
  2. El selector manual para "otras" ahora incluye no solo las Obras Sociales (OS) sino también todas las prepagas disponibles. Se excluyen las OS que el paciente ya posee.
  3. Se agrega el componente en la pantalla -Dar Turno- en ventanilla de citas.
  4. Se reemplaza el componente en la solapa -Turnos- en ventanilla de citas.
  5. Se reemplaza el componente al seleccionar turno, para prestaciones iniciadas, en -Buscador de Turnos-.

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

Requiere actualizaciones en andes-test-integracion

@ma7payne ma7payne requested review from a team as code owners April 24, 2023 15:45
@ma7payne ma7payne requested review from MCele and negro89 April 24, 2023 15:45
@MarianoCampetella
Copy link
Contributor

MarianoCampetella commented May 5, 2023 •
edited
Loading

Cuando agrego un sobreturno me sigue saliendo igual a como estaba antes.
image

Otra cosa a corregir es en el buscador de turnos y prestaciones. Cuando selecciono un turno cuyo paciente tiene obra social me sale el select como se pide en la tarea. Pero cuando selecciono un item del listado con obra social SUMAR y luego selecciono otro donde tiene cualquier obra social distinta a SUMAR ocurre lo que se ve en la imagen, es decir, no sale el select como corresponde y queda la leyenda de SUMAR solamente
image

Ya que estamos aprovechar a resolver los conflictos y correr los test!

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label May 5, 2023
@ma7payne ma7payne force-pushed the CIT-214 branch 3 times, most recently from 9786f6e to e3d6f7e Compare May 12, 2023 15:15
@ma7payne ma7payne force-pushed the CIT-214 branch 2 times, most recently from 68ac728 to a269f70 Compare May 16, 2023 11:54
@ma7payne
Copy link
Contributor Author

@MarianoCampetella Están listos los cambios para volver a chequear el selector, de no poder verlo, revisar la cache del navegador. El PR esta al día y sin conflictos, quedo atento.

@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels May 16, 2023
@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 8901
CYPRESS RUN: 8066
TEST START: 2023-05-16T12:46:09.726Z
TOTAL: 323
SUCCESS: 304
FAIL: 17
SKIPPED: 2
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@andes andes deleted a comment from silviroa May 16, 2023
@andes andes deleted a comment from silviroa May 16, 2023
@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 8908
CYPRESS RUN: 8073
TEST START: 2023-05-17T11:46:28.863Z
TOTAL: 406
SUCCESS: 402
FAIL: 2
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 10072
CYPRESS RUN: 9180
TEST START: 2023-12-19T16:30:33.335Z
TOTAL: 96
SUCCESS: 0
FAIL: 96
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@silviroa silviroa added test fail and removed test ok Los test estan ok labels Dec 19, 2023
@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 10077
CYPRESS RUN: 9185
TEST START: 2023-12-19T18:02:02.920Z
TOTAL: 405
SUCCESS: 402
FAIL: 1
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 10079
CYPRESS RUN: 9187
TEST START: 2023-12-19T18:22:21.137Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

@silviroa silviroa added test ok Los test estan ok and removed test fail labels Dec 19, 2023
@MCele MCele removed the Aprobado label Dec 21, 2023
@MCele
Copy link
Contributor

MCele commented Dec 21, 2023

Cambios pedido en api

@ma7payne ma7payne force-pushed the CIT-214 branch 2 times, most recently from 869acbb to eebceba Compare December 22, 2023 11:57
@MCele MCele added the Aprobado label Dec 22, 2023
@MCele MCele merged commit 1be5f4b into master Dec 22, 2023
2 checks passed
@MCele MCele deleted the CIT-214 branch December 22, 2023 15:26
liquid36 pushed a commit that referenced this pull request Dec 27, 2023
@semantic-release-bot
chore(release): 5.130.0 [skip ci]
1088457 
# [5.130.0](v5.129.0...v5.130.0) (2023-12-27)

### Bug Fixes

* **RUP:** modifica label en boton de filtro ([#2966](#2966)) ([9d01cee](9d01cee))
* **TYP-10:** Corregir Acceso al buscador ([#2964](#2964)) ([a49ed8d](a49ed8d))

### Features

* **CIT:** implementa selector de financiadores ([#2854](#2854)) ([1be5f4b](1be5f4b))
* **RUP:** visualización de botones del buscador ([#2933](#2933)) ([0f0eb26](0f0eb26))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AgosLizzi AgosLizzi AgosLizzi approved these changes

@MarianoCampetella MarianoCampetella MarianoCampetella approved these changes

@Fabio-Ramirez Fabio-Ramirez Fabio-Ramirez approved these changes

@MCele MCele MCele approved these changes

@negro89 negro89 Awaiting requested review from negro89 negro89 is a code owner automatically assigned from andes/code-review

Assignees
No one assigned
Labels
Aprobado prioritario Tarea con prioridad alta test ok Los test estan ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@ma7payne @MarianoCampetella @silviroa @negro89 @MCele @Fabio-Ramirez @AgosLizzi