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

[10.0][MIG] account_invoice_refund_link #254

Merged
merged 3 commits into from
Apr 28, 2017

Conversation

luismontalba
Copy link
Member

@luismontalba luismontalba commented Apr 10, 2017

Link refund invoice with original

This module links customer and supplier refunds with the invoice that originate
them (and invoice lines).

Usage

This module creates in the invoice form a new 'Refund' page.

In case of refund invoices this new page shows refund reason and original
invoice information.

In case of original invoices this new page shows a line for each refund
invoice with main information.

cc @Tecnativa

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Squashing last 2 commits together when merging.

Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

tested on runbot

pedrobaeza and others added 3 commits April 28, 2017 18:44
Odoo already includes a method for preparing the refund, so using it
we don't have to make any guess about the returned domain and the
method also serves for refunds created directly by code.

Initial line match has also been improved comparing product or
description.

---

Remove not needed migration script
@pedrobaeza pedrobaeza force-pushed the 10.0-mig-account_invoice_refund_link branch from 0d1ca44 to e197142 Compare April 28, 2017 16:45
@pedrobaeza pedrobaeza merged commit 0480221 into OCA:10.0 Apr 28, 2017
@pedrobaeza pedrobaeza deleted the 10.0-mig-account_invoice_refund_link branch April 28, 2017 16:54
@pedrobaeza pedrobaeza mentioned this pull request Apr 28, 2017
34 tasks
@MiquelRForgeFlow
Copy link
Contributor

@pedrobaeza @luismontalba
Some tests of this module are failing in the Travis of #238.
Please, check and say what to do!

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.

5 participants