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

Improvement - Validación de importación - Eliminar validación innecesaria del proceso de validación de importación #511

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Improvement - Validación de importación - Eliminar validación innecesaria del proceso de validación de importación #511

merged 1 commit into from
Dec 19, 2024

Conversation

ManuSinergiaCRM
Copy link
Collaborator

Descripción

Este PR implementa la solución propuesta en el issue relacionado.

Pruebas

  1. Crear una persona con Tipo de identificación y Número de identificación
  2. Exportarlo y eliminar todas las columnas menos las de Apellido, Tipo de identificación y Número de identificación
  3. Eliminar el valor del campo Tipo de identificación
  4. Realizar la validación del fichero de importación y comprobar que el proceso indica que el registro ha sido validado correctamente.
  5. Duplicar el fichero y eliminar la columna de Número de identificación.
  6. Realizar la validación del fichero de importación y comprobar que el proceso indica que el registro ha sido validado correctamente.
  7. Importar el fichero y comprobar que el CRM, durante la importación, ha indicado el campo Tipo de identificación aunque en el fichero de importación no estuviera indicado.

@ManuSinergiaCRM ManuSinergiaCRM added bug Something isn't working Importación labels Dec 9, 2024
@ManuSinergiaCRM ManuSinergiaCRM self-assigned this Dec 9, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 9, 2024 •
edited
Loading

Actions executed at: 2024-12-19 08:20:50.

enricsinergia
enricsinergia approved these changes Dec 16, 2024
View reviewed changes
Copy link
Collaborator

@enricsinergia enricsinergia left a comment

Choose a reason for hiding this comment

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

(A)probado

ainaraRT
ainaraRT approved these changes Dec 17, 2024
View reviewed changes
Copy link
Collaborator

@ainaraRT ainaraRT left a comment

Choose a reason for hiding this comment

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

(A)Probado

@AlbertoSTIC AlbertoSTIC merged commit 3257138 into develop Dec 19, 2024
1 check passed
@AlbertoSTIC AlbertoSTIC deleted the bug/RemoveUnnecessaryValidation branch December 19, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ainaraRT ainaraRT ainaraRT approved these changes

@enricsinergia enricsinergia enricsinergia approved these changes

Assignees

@ManuSinergiaCRM ManuSinergiaCRM

Labels
bug Something isn't working Importación
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Incidencia - Validación de importación - Eliminar validación no necesaria
4 participants
@ManuSinergiaCRM @enricsinergia @ainaraRT @AlbertoSTIC