Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
L
l10n_mx_facturae
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 6
    • Issues 6
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 4
    • Merge Requests 4
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • l10n_mx
  • l10n_mx_facturae
  • Merge Requests
  • !145

Merged
Opened 2 years ago by Antonio Manzano Romero@antonio.manzanoContributor

fix(account.voucher): se agrega redondeo para el campo TotalTrasladosBaseIVA

  • Overview 12
  • Commits 1
  • Pipelines 4
  • Changes 1
All threads resolved
  • Loading...

Se agregó el redondeo para el campo TotalTrasladosBaseIVA, esto se realizó porque al momento de timbrar un pago se generaba la siguiente traza de error: RP20204 El valor del campo TotalTrasladosBaseIVA16 no es igual al redondeo de la suma del resultado de multiplicar cada uno de los importes de los atributos BaseP de los impuestos trasladados registrados en el elemento TrasladoP donde los atributos contengan en ImpuestoP el valor IVA, en TipoFactorP el valor Tasa y en TasaOCuotaP el valor 0.160000, por el valor registrado en el atributo TipoCambioP de cada nodo Pago.

Request to merge fix_round_tax into master
  • Download as
  • Email patches

  • Plain diff

Pipeline #16538 passed for 22aca5e5 on fix_round_tax
Checking approval status

Merged by Agustín Cruz 2 years ago (Jun 15, 2022 6:22pm GMT+0000) 2 years ago

The changes were merged into master with da624e4f

The source branch has been deleted

Pipeline #16539 passed for da624e4f on master

    • Agustín Cruz
      Agustín Cruz @agb80 · 2 years ago
      Owner
      Resolved by Agustín Cruz 2 years ago

      @antonio.manzano un favor, ¿me puedes apoyar agregando el redondeo a todos los campos para curarnos en salud y no esperar a que se presente otro error en productivo:

      • TotalTrasladosBaseIVAExento
      • TotalTrasladosImpuestoIVA
      • TotalRetencionesIVA
      • TotalRetencionesISR
      • TotalRetencionesIEPS
      • ImporteDR
      • ImporteP

      También te pediría que agregaras igualmente el redondeo en los pagos realizados en moneda extranjera:

      • https://gitlab.openpyme.mx/l10n_mx/l10n_mx_facturae/-/blob/a42c0834dec423573f6a7abc26c3aa26b90608be/l10n_mx_facturae/templates/account_voucher.txt#L98
      • https://gitlab.openpyme.mx/l10n_mx/l10n_mx_facturae/-/blob/a42c0834dec423573f6a7abc26c3aa26b90608be/l10n_mx_facturae/templates/account_voucher.txt#L112
    • Antonio Manzano Romero Agustín Cruz
      Last reply by Agustín Cruz 2 years ago
  • Antonio Manzano Romero @antonio.manzano added 1 commit 2 years ago

    added 1 commit

    • f59b8a46 - fix(account.voucher): se agrega redondeo para el campo TotalTrasladosBaseIVA

    Compare with previous version

  • Agustín Cruz
    Agustín Cruz @agb80 started a thread on an old version of the diff 2 years ago
    Automatically resolved by Antonio Manzano Romero 2 years ago
  • Agustín Cruz
    Agustín Cruz @agb80 started a thread on an old version of the diff 2 years ago
    Automatically resolved by Antonio Manzano Romero 2 years ago
  • Agustín Cruz
    Agustín Cruz @agb80 started a thread on an old version of the diff 2 years ago
    Automatically resolved by Antonio Manzano Romero 2 years ago
  • Agustín Cruz
    Agustín Cruz @agb80 started a thread on an old version of the diff 2 years ago
    Automatically resolved by Antonio Manzano Romero 2 years ago
  • Agustín Cruz
    Agustín Cruz @agb80 started a thread on an old version of the diff 2 years ago
    Automatically resolved by Antonio Manzano Romero 2 years ago
  • Agustín Cruz
    Agustín Cruz @agb80 started a thread on an old version of the diff 2 years ago
    Automatically resolved by Antonio Manzano Romero 2 years ago
  • Agustín Cruz
    Agustín Cruz @agb80 started a thread on an old version of the diff 2 years ago
    Automatically resolved by Antonio Manzano Romero 2 years ago
  • Agustín Cruz
    Agustín Cruz @agb80 started a thread on an old version of the diff 2 years ago
    Automatically resolved by Antonio Manzano Romero 2 years ago
  • Antonio Manzano Romero @antonio.manzano added 1 commit 2 years ago

    added 1 commit

    • c8ae8196 - fix(account.voucher): se agrega redondeo para el campo TotalTrasladosBaseIVA

    Compare with previous version

  • Agustín Cruz
    Agustín Cruz @agb80 started a thread on an old version of the diff 2 years ago
    Automatically resolved by Antonio Manzano Romero 2 years ago
  • Antonio Manzano Romero @antonio.manzano resolved all threads 2 years ago

    resolved all threads

  • Antonio Manzano Romero @antonio.manzano added 1 commit 2 years ago

    added 1 commit

    • 22aca5e5 - fix(account.voucher): se agrega redondeo para el campo TotalTrasladosBaseIVA

    Compare with previous version

  • Agustín Cruz @agb80 merged 2 years ago

    merged

  • You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
Please register or sign in to reply
0 Assignees
Assign to
0 Reviewers
Request review from
None
Milestone
None
Assign milestone
None
Time tracking
No estimate or time spent
0
Labels
None
Assign labels
  • No matching results
  • Manage project labels
Lock merge request
Unlocked
participants
Reference: l10n_mx/l10n_mx_facturae!145
Source branch: fix_round_tax

    0 pending comments