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

[8.0] [IMP] [account_due_list] add the name of the move to the list view #108

Merged
merged 2 commits into from
Nov 2, 2016

Conversation

JordiBForgeFlow
Copy link
Member

Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
Solution: Add the move line name to this list, as it provides valuable info.

Modules such as https://github.com/OCA/bank-statement-reconcile/tree/8.0/account_invoice_reference even make it more evident the need to show the move line name.

 Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
@@ -13,6 +13,7 @@
<field name="stored_invoice_id" readonly="1"/>
<field name="invoice_date" readonly="1"/>
<field name="invoice_origin" readonly="1"/>
<field name="name" readonly="1" attrs=""/>
Copy link
Member

Choose a reason for hiding this comment

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

Why attrs=""?

@JordiBForgeFlow
Copy link
Member Author

oops, now fixed ;)

Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow 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 + code review 👍

@JordiBForgeFlow
Copy link
Member Author

cc @fclementic2c Do you agree on this one? (Quick to validate)

@andhit-r
Copy link
Member

andhit-r commented Nov 2, 2016

Code Review: 👍

@pedrobaeza pedrobaeza merged commit fd39e3f into OCA:8.0 Nov 2, 2016
JordiBForgeFlow added a commit to ForgeFlow/account-payment that referenced this pull request Dec 28, 2016
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
JordiBForgeFlow added a commit to ForgeFlow/account-payment that referenced this pull request Jan 18, 2017
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
pedrobaeza pushed a commit to ForgeFlow/account-payment that referenced this pull request Mar 23, 2017
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
eLBati pushed a commit to eLBati/account-payment that referenced this pull request Mar 28, 2017
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
vonpupp pushed a commit to vonpupp/account-payment that referenced this pull request Mar 29, 2017
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
ozono pushed a commit to ozono/account-payment that referenced this pull request Jan 30, 2018
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
pedrobaeza pushed a commit to ozono/account-payment that referenced this pull request Mar 21, 2018
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
eLBati pushed a commit to eLBati/account-payment that referenced this pull request Oct 29, 2018
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
MiquelRForgeFlow pushed a commit to ForgeFlow/account-payment that referenced this pull request Jan 29, 2020
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
SimoneVagile pushed a commit to SimoneVagile/account-payment that referenced this pull request Mar 30, 2021
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
Freni-OSI pushed a commit to ursais/account-payment that referenced this pull request Apr 2, 2021
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
LudLaf pushed a commit to Tecnativa/account-payment that referenced this pull request Apr 12, 2022
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
mileo pushed a commit to kmee/account-payment that referenced this pull request Oct 21, 2022
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
ramiadavid pushed a commit to ramiadavid/account-payment that referenced this pull request Oct 21, 2022
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
luisDIXMIT pushed a commit to dixmit/account-payment that referenced this pull request Mar 13, 2024
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
ramiadavid pushed a commit to ramiadavid/account-payment that referenced this pull request Oct 10, 2024
Problem: Key information, such as the supplier invoice number, does not appear in the due list. The information that appears in the list is inconsistent with the data shown in the list of move lines. The name of the move line is missing.
 Solution: Add the move line name to this list, as it provides valuable info.
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.

4 participants