-
-
Notifications
You must be signed in to change notification settings - Fork 177
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] Add field 'UNECE Due date' on taxes and corresponding nomenclature #26
Conversation
…equired for Factur-X support) Improve search view on unece code list.
…lues; we decide to use only 1 for invoice generation and we have a hack on import
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change version number
"fiscal administration. For a purchase VAT tax, it is the date " | ||
"on which that VAT can be deducted.") | ||
unece_due_date_code = fields.Char( | ||
related='unece_due_date_id.code', store=True, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make it readonly=True
or you will have undesired writes.
'unece.code.list', string='UNECE Due Date', | ||
domain=[('type', '=', 'date')], ondelete='restrict', | ||
help="Select the due date of that tax from the official " | ||
"nomenclature of the United Nations Economic " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Align this at the level of the first "
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
flake8 is happy with it. I think it's a matter of taste here...
…needed for Factur-X standard)
I got the info that probably other changes in nomenclature part of the Factur-X standard will take place. So this PR should be merged when the final version of the standard is published (scheduled for September 30th) |
…ded recently by the Factur-X standard
This is now ready to be merged. I updated all the UNECE nomenclatures and the Factur-X standard has been frozen. |
Conflict resolved. Ready to merge. |
This is required for Factur-X support, cf this PR OCA/edi#22
Improve search view on unece code list.