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

Enhancement - Economía - Generación de Remesas multi-entidad #457

Merged
merged 11 commits into from
Jan 21, 2025

Conversation

jordiSTIC
Copy link
Collaborator

@jordiSTIC jordiSTIC commented Oct 24, 2024

Descripción

En este PR se propone una solución para poder generar remesas desde SinergiaCRM en nombre de más de una entidad.

Hasta el momento, únicamente era posible generar las remesas en nombre de la propia entidad, según lo definido en las configuraciones:

  • GENERAL_ORGANIZATION_NAME: Nombre de la entidad usuaria del CRM (origen de la remesa)
  • SEPA_TRANSFER_DEFAULT_REMITTANCE_INFO
  • SEPA_TRANSFER_DEBITOR_IDENTIFIER
  • SEPA_DEBIT_DEFAULT_REMITTANCE_INFO
  • SEPA_DEBIT_CREDITOR_IDENTIFIER
  • SEPA_DEBIT_BIC_MODE
  • SEPA_BIC_CODE

Solución propuesta

Se propone la definición de las entidades y los parámetros necesarios para la generación de remesas mediante la creación de nuevas configuraciones, añadiendo un sufijo a estas que identifiquen a la entidad. De esta forma, las configuraciones tendrán la forma: CONFIGURACION_XXXX, siendo XXXX el identificador de la posible entidad emisora de remesas. Por ejemplo:

  • GENERAL_ORGANIZATION_NAME_ENTIDAD2
  • SEPA_TRANSFER_DEFAULT_REMITTANCE_INFO_ENTIDAD2
  • [...]

En el módulo de remesas se ha incluido el nuevo campo 'origin_organization' que contendrá el identificador de la entidad orígen de la remesa; esto es, el sufijo usado. Por defecto estará vacío, significará que las configuraciones a usar no tendrán sufijo: como hasta ahora.

En la vista de edición del módulo de Remesas, se ha incluído un selector de la organización origen. Este selector muestra los distintos valores de las configuraciones GENERAL_ORGANIZATION_NAME, incluyendo las que tienen sufijo

Pruebas

Generales

  1. En Administración - SinergiaCRM - Configuración, duplicar la configuración GENERAL_ORGANIZATION_NAME y añadir el sufijo para otra entidad (XXXX u otro identificador) con un nombre de entidad
  2. En Administración - Editor de Listas Desplegables - stic_remittances_ibans_list, verificar que al menos una cuenta sea válida
  3. Crear una remesa (módulo Remesas)
  4. Verificar que aparece el selector "Organización de origen" con las organizaciones definidas

Remesas de recibos

  1. En Administración - SinergiaCRM - Configuración, duplicar las configuraciones SEPA_DEBIT_CREDITOR_IDENTIFIER y SEPA_DEBIT_DEFAULT_REMITTANCE_INFO, añadiéndole el sufijo de la otra entidad (XXXX u otro identificador) con un valor
  2. Crear una remesa del tipo "Domicialiaciones", seleccionando la nueva entidad como origen
  3. Añadir Pagos a la remesa (subpanel Pagos)
  4. Acciones - Generar remesa SEPA de recibos
  5. Verificar que el XML generado contiene los datos de la nueva entidad

Remesas de tranferencias

  1. En Administración - SinergiaCRM - Configuración, duplicar las configuraciones SEPA_TRANSFER_DEBITOR_IDENTIFIER y SEPA_TRANSFER_DEFAULT_REMITTANCE_INFO, añadiéndole el sufijo de la otra entidad (XXXX u otro identificador) con un valor
  2. Crear una remesa del tipo "Tranferencias", seleccionando la nueva entidad como origen
  3. Añadir Pagos a la remesa (subpanel Pagos)
  4. Acciones - Generar remesa SEPA de tranferencias
  5. Verificar que el XML generado contiene los datos de la nueva entidad

@jordiSTIC jordiSTIC self-assigned this Oct 24, 2024
@jordiSTIC jordiSTIC added enhancement New feature or request Economía This issue or pull request already exists labels Oct 24, 2024
Copy link

github-actions bot commented Oct 24, 2024

Actions executed at: 2025-01-21 09:08:11.

@jordiSTIC jordiSTIC marked this pull request as ready for review October 25, 2024 09:32
@jordiSTIC jordiSTIC requested a review from jalbaiges October 25, 2024 10:31
Copy link

@jalbaiges jalbaiges left a comment

Choose a reason for hiding this comment

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

Cadenas revisadas.

Sugerencias:

  1. Cambiar en el código origin_organization por issuing_organization.

  2. Hacer el campo auditado.

@jordiSTIC
Copy link
Collaborator Author

Sugerencias:

  1. Cambiar en el código origin_organization por issuing_organization.
  2. Hacer el campo auditado.

Cambios realizados

@jordiSTIC jordiSTIC requested a review from jalbaiges October 28, 2024 14:45
jalbaiges
jalbaiges previously approved these changes Oct 28, 2024
Copy link

@jalbaiges jalbaiges left a comment

Choose a reason for hiding this comment

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

Aprobado en cuanto a cadenas y en relación a los cambios sugeridos. No se ha probado el funcionamiento.

juanSTIC

This comment was marked as resolved.

juanSTIC

This comment was marked as resolved.

juanSTIC

This comment was marked as resolved.

@jordiSTIC
Copy link
Collaborator Author

Se han corregido los errores en las vistas emergentes: no se incluía la referencia al fichero de definición necesario

@jordiSTIC jordiSTIC requested a review from juanSTIC January 16, 2025 11:33
@AlbertoSTIC AlbertoSTIC removed their request for review January 20, 2025 10:01
Copy link
Collaborator

@juanSTIC juanSTIC left a comment

Choose a reason for hiding this comment

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

Visto y probado

ManuSinergiaCRM

This comment was marked as spam.

@ManuSinergiaCRM ManuSinergiaCRM self-requested a review January 20, 2025 15:10
Copy link
Collaborator

@ManuSinergiaCRM ManuSinergiaCRM 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

@enricsinergia enricsinergia merged commit 264c036 into develop Jan 21, 2025
1 check passed
@enricsinergia enricsinergia deleted the enhancement/remittancesMultiOrigin branch January 21, 2025 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Economía This issue or pull request already exists enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mejora - Economía - Generar Remesa multi-entidad
5 participants