Tryton - Issues

 

Issue2329

Title Add tax legal advise
Priority feature Status resolved
Superseder Nosy List ced, pokoli, reviewbot, roundup-bot
Type behavior Components account, account_invoice
Assigned To pokoli Keywords review
Reviews 35541002, 41541002
View: 35541002, 41541002

Created on 2011-12-06.17:29:27 by ced, last changed by roundup-bot.

Files
File name Uploaded Type Edit Remove
invoice.odt pokoli, 2017-08-11.12:05:21 application/vnd.oasis.opendocument.text
invoice.odt pokoli, 2017-08-16.10:12:27 application/vnd.oasis.opendocument.text
Messages
New changeset e00039f512ae by Sergi Almacellas Abellana in branch 'default':
Show taxes extra description on invoice report
http://hg.tryton.org/modules/account_invoice/rev/e00039f512ae
New changeset 91ef3a4a19f5 by Sergi Almacellas Abellana in branch 'default':
Add legal notice on taxes
http://hg.tryton.org/modules/account/rev/91ef3a4a19f5
review41541002 updated at https://codereview.tryton.org/41541002/#ps120001
review41541002 updated at https://codereview.tryton.org/41541002/#ps100001
review41541002 updated at https://codereview.tryton.org/41541002/#ps80001
review41541002 updated at https://codereview.tryton.org/41541002/#ps60001
review41541002 updated at https://codereview.tryton.org/41541002/#ps40001
review35541002 updated at https://codereview.tryton.org/35541002/#ps100001
review35541002 updated at https://codereview.tryton.org/35541002/#ps80001
review41541002 updated at https://codereview.tryton.org/41541002/#ps20001
review35541002 updated at https://codereview.tryton.org/35541002/#ps60001
New review41541002 at https://codereview.tryton.org/41541002/#ps1
review35541002 updated at https://codereview.tryton.org/35541002/#ps40001
msg35494 (view) Author: [hidden] (pokoli) (Tryton committer) (Tryton translator) Date: 2017-09-04.09:59:54
As we now have fodt support, I added review41541002 for the account invoice module
msg35169 (view) Author: [hidden] (pokoli) (Tryton committer) (Tryton translator) Date: 2017-08-16.10:12:27
I uploaded the review and the invoice module taking in account msg35145
review35541002 updated at https://codereview.tryton.org/35541002/#ps20001
msg35145 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-08-12.14:58:10
I'm not fan of the name "legal advise". I think it about "description_extra". And I also think it should be a Text field.
For the invoice, I would put the text inside the tax table to clearly show it is linked to the taxes. Also it should show only once the extra description per taxes. And maybe even if two different taxes have the same text, it should be displayed only once.
msg35126 (view) Author: [hidden] (pokoli) (Tryton committer) (Tryton translator) Date: 2017-08-11.12:05:08
I attached the invoice template which shows the taxes legal information when the tax have it defined. 

We are now ready for testing.
review35541002 updated at https://codereview.tryton.org/35541002/#ps1
msg35119 (view) Author: [hidden] (pokoli) (Tryton committer) (Tryton translator) Date: 2017-08-10.17:58:54
I added review35541002 which adds the field on the account module. 

I will upload a modified version of the invoice file to show this information.
msg9479 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2011-12-06.17:29:27
There is many requirements to be able to put a legal statement on invoice
depending of the taxes used.

https://groups.google.com/d/topic/tryton/L248G7JVKFo/discussion
History
Date User Action Args
2017-10-02 09:46:33roundup-botsetmessages: + msg35987
2017-10-02 09:46:24roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg35986
2017-09-29 14:44:35reviewbotsetmessages: + msg35934
2017-09-28 16:12:46reviewbotsetmessages: + msg35909
2017-09-26 11:38:30reviewbotsetmessages: + msg35789
2017-09-25 12:09:47reviewbotsetmessages: + msg35761
2017-09-22 14:47:35reviewbotsetmessages: + msg35742
2017-09-22 14:24:47reviewbotsetmessages: + msg35741
2017-09-22 12:25:48reviewbotsetmessages: + msg35738
2017-09-18 18:15:15reviewbotsetmessages: + msg35678

Showing 10 items. Show all history (warning: this could be VERY long)