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

fix: now logs show details and the inputs that caused the error #302

Merged
merged 10 commits into from
Nov 5, 2024

Conversation

XavierLopez25
Copy link
Collaborator

Jira

Link al issue en Jira

Descripción

Se cambiaron los logs de pino, para que se muestre el error y los parámetros que hicieron que fallara.

Tasks

  • Asignate a tí mismo dentro de la PR.
  • Asegúrate que el nombre de la PR siga el formato de Conventional Commits.
  • Añade una breve descripción de la escencia de tus cambios en tu PR.
  • Agrega el link al issue de Jira.
  • Asegurate que la PR pase todos los chequeos de CI.
  • Ponle las tags correspondientes a tu PR.
    Backend: PRs que modifican lógica relacionada al backend.
    Frontend: PRs que modifican lógica relacionada al frontend.
    Database: PRs que modifican lógica relacionada a la base de datos.
    Wiki: PRs que editan la wiki.
    Nix: PRs que modifican el entorno de desarrollo en Nix.
    CI/CD: PRs relacionadas con la CI/CD pipeline.

@XavierLopez25 XavierLopez25 marked this pull request as ready for review November 1, 2024 01:08
Copy link
Collaborator

@ElrohirGT ElrohirGT left a comment

Choose a reason for hiding this comment

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

Mi review del GetAllergic aplica para todos los demás! Porfa ahí se los pones jaja

Copy link
Collaborator

@ElrohirGT ElrohirGT left a comment

Choose a reason for hiding this comment

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

@XavierLopez25 XavierLopez25 merged commit d5d5c03 into develop Nov 5, 2024
3 checks passed
@XavierLopez25 XavierLopez25 deleted the fix/fix-pino-logs-and-logout-button branch November 5, 2024 22:28
ElrohirGT added a commit that referenced this pull request Nov 9, 2024
* fix: max length on general information view input (#300)

* fix: changed area and carrera columns to length 100

* fix: added maxlength to inputs

* fix: lint and format

* fix: fixed id

* fix: fixing non pathologic message and use memo hook in views (#301)

* fix: adding birthday validate form

* fix: eliminate blood visualization and adding a toast error

* fix: fixing message in welcome view

* fix: fixing placeholders and padding in buttons

* fix: format and lint

* feat: adding UseMemo in History doctor

* fix: format and lint

* feat: adding memo in views

* fix: separate useMemo in general info view

* fix: change Seguro to Seguro Médico (#303)

* feat: comments and access (#304)

* fix: Added Protected API ID

* fix: now logs show details and the inputs that caused the error (#302)

* fix: now logs show details and the inputs that caused the error

* fix: fixed button on all sections

* feat: added toSafeEvent function

* fix: now 500 status code errors have more useful info

* fix: restore accidental delete

* fix: Fixed hasOwn on old browsers (#306)

* fix: UpdatePatientGeneralInformation navigation (#307)

* fix: UpdatePatientGeneralInformation navigation

* feat: Improved navigation for all views

* feat: error handlers for toasts (#305)

* feat: error handlers for toasts

* fix: path typo

* fix: formatter missed

* fix: Personal styles

---------

Co-authored-by: Xavier López (Nathan) <[email protected]>
Co-authored-by: Nahomy Castro <[email protected]>
Co-authored-by: Bianca Calderón <[email protected]>
Co-authored-by: DanielDubon <[email protected]>
ElrohirGT added a commit that referenced this pull request Nov 11, 2024
* fix: max length on general information view input (#300)

* fix: changed area and carrera columns to length 100

* fix: added maxlength to inputs

* fix: lint and format

* fix: fixed id

* fix: fixing non pathologic message and use memo hook in views (#301)

* fix: adding birthday validate form

* fix: eliminate blood visualization and adding a toast error

* fix: fixing message in welcome view

* fix: fixing placeholders and padding in buttons

* fix: format and lint

* feat: adding UseMemo in History doctor

* fix: format and lint

* feat: adding memo in views

* fix: separate useMemo in general info view

* fix: change Seguro to Seguro Médico (#303)

* feat: comments and access (#304)

* fix: Added Protected API ID

* fix: now logs show details and the inputs that caused the error (#302)

* fix: now logs show details and the inputs that caused the error

* fix: fixed button on all sections

* feat: added toSafeEvent function

* fix: now 500 status code errors have more useful info

* fix: restore accidental delete

* fix: Fixed hasOwn on old browsers (#306)

* fix: UpdatePatientGeneralInformation navigation (#307)

* fix: UpdatePatientGeneralInformation navigation

* feat: Improved navigation for all views

* feat: error handlers for toasts (#305)

* feat: error handlers for toasts

* fix: path typo

* fix: formatter missed

* fix: Personal styles

* fix: Register user popup on mobile

* fix: Scrolling white background

---------

Co-authored-by: Xavier López (Nathan) <[email protected]>
Co-authored-by: Nahomy Castro <[email protected]>
Co-authored-by: Bianca Calderón <[email protected]>
Co-authored-by: DanielDubon <[email protected]>
ElrohirGT added a commit that referenced this pull request Nov 12, 2024
* fix: max length on general information view input (#300)

* fix: changed area and carrera columns to length 100

* fix: added maxlength to inputs

* fix: lint and format

* fix: fixed id

* fix: fixing non pathologic message and use memo hook in views (#301)

* fix: adding birthday validate form

* fix: eliminate blood visualization and adding a toast error

* fix: fixing message in welcome view

* fix: fixing placeholders and padding in buttons

* fix: format and lint

* feat: adding UseMemo in History doctor

* fix: format and lint

* feat: adding memo in views

* fix: separate useMemo in general info view

* fix: change Seguro to Seguro Médico (#303)

* feat: comments and access (#304)

* fix: Added Protected API ID

* fix: now logs show details and the inputs that caused the error (#302)

* fix: now logs show details and the inputs that caused the error

* fix: fixed button on all sections

* feat: added toSafeEvent function

* fix: now 500 status code errors have more useful info

* fix: restore accidental delete

* fix: Fixed hasOwn on old browsers (#306)

* fix: UpdatePatientGeneralInformation navigation (#307)

* fix: UpdatePatientGeneralInformation navigation

* feat: Improved navigation for all views

* feat: error handlers for toasts (#305)

* feat: error handlers for toasts

* fix: path typo

* fix: formatter missed

* fix: Personal styles

* fix: Register user popup on mobile

* fix: Scrolling white background

* fix: RegisterView popup

---------

Co-authored-by: Xavier López (Nathan) <[email protected]>
Co-authored-by: Nahomy Castro <[email protected]>
Co-authored-by: Bianca Calderón <[email protected]>
Co-authored-by: DanielDubon <[email protected]>
ElrohirGT added a commit that referenced this pull request Nov 12, 2024
* fix: max length on general information view input (#300)

* fix: changed area and carrera columns to length 100

* fix: added maxlength to inputs

* fix: lint and format

* fix: fixed id

* fix: fixing non pathologic message and use memo hook in views (#301)

* fix: adding birthday validate form

* fix: eliminate blood visualization and adding a toast error

* fix: fixing message in welcome view

* fix: fixing placeholders and padding in buttons

* fix: format and lint

* feat: adding UseMemo in History doctor

* fix: format and lint

* feat: adding memo in views

* fix: separate useMemo in general info view

* fix: change Seguro to Seguro Médico (#303)

* feat: comments and access (#304)

* fix: Added Protected API ID

* fix: now logs show details and the inputs that caused the error (#302)

* fix: now logs show details and the inputs that caused the error

* fix: fixed button on all sections

* feat: added toSafeEvent function

* fix: now 500 status code errors have more useful info

* fix: restore accidental delete

* fix: Fixed hasOwn on old browsers (#306)

* fix: UpdatePatientGeneralInformation navigation (#307)

* fix: UpdatePatientGeneralInformation navigation

* feat: Improved navigation for all views

* feat: error handlers for toasts (#305)

* feat: error handlers for toasts

* fix: path typo

* fix: formatter missed

* fix: Personal styles

* fix: Register user popup on mobile

* fix: Scrolling white background

* fix: RegisterView popup

---------

Co-authored-by: Xavier López (Nathan) <[email protected]>
Co-authored-by: Nahomy Castro <[email protected]>
Co-authored-by: Bianca Calderón <[email protected]>
Co-authored-by: DanielDubon <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants