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
  • !88

Merged
Opened 6 years ago by Agustín Cruz@agb80Owner

fix(account.invoice): mantiene la relación entre la factura y su CFDI cuando se cancela

  • Overview 2
  • Commits 1
  • Changes 2
1 unresolved thread
  • Loading...

Cuando una factura se cancela no tiene porque perder la relación entre la factura y el CFDI correspondiente.

cc @OpenPymeMx

Request to merge agb80:fix/2.9.3 into master
  • Download as
  • Email patches

  • Plain diff

Checking pipeline status.

Checking approval status

Merged by Agustín Cruz 6 years ago (Apr 23, 2019 5:34pm GMT+0000) 6 years ago

The changes were merged into master with e076f96c

The source branch has been deleted

Could not retrieve the pipeline status. For troubleshooting steps, read the documentation.


  • Agustín Cruz @agb80 added Bug label 6 years ago

    added Bug label

  • Agustín Cruz @agb80 mentioned in issue facturacion#113 (closed) 6 years ago

    mentioned in issue facturacion#113 (closed)

  • Rivera Santos
    Rivera Santos @santos.rivera · 6 years ago
    Contributor

    ¿Va a ser algo parecido a los complementos de pago?

  • Salvador Martínez
    Salvador Martínez @chavamm.83 started a thread on the diff 6 years ago
    l10n_mx_facturae/models/account_voucher.py
    33 33 """Cancel CFDI for selected vouchers"""
    34 34 res = super(AccountVoucher, self).cancel_voucher()
    35 35 self.cancel_cfdi()
    36 self.unlink_cfdi()
    36 # Delete relation from voucher and CFDI
    37 self.cfdi_id = False
    • Salvador Martínez
      Salvador Martínez @chavamm.83 · 6 years ago
      Developer

      Saludos @agb80 Me parece bien que quede asi, que en cada tipo de cfdi se quite la relacion con el cfdi_id si asi fuera necesario, y en el que no sea asi, que se deje la relacion.

      Como te comentaba, que en el caso de los cfdi de las Facturas, ahi sería conveniente mantener la relacion, ya que se puede necesitar en un futuro requerirla.

      Saludos!

    • Please register or sign in to reply
  • Agustín Cruz @agb80 merged 6 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
1
Labels
Bug
Assign labels
  • No matching results
  • Manage project labels
Lock merge request
Unlocked
participants
Reference: l10n_mx/l10n_mx_facturae!88
Source branch: fix/2.9.3

    0 pending comments