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

[IMP] account, l10n_be_invoice_bba: update invoice reference fields #29858

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@william-andre
Copy link
Contributor

william-andre commented Jan 2, 2019

OPW 1920906

l10n_be_invoice_bba:
remove the l10n_be_structured_comm field
'random'/'date' references will not be longer used
new structured_invoice_number and structured_partner_ref reference types

account:
new rf_invoice_number and rf_partner_ref added as possible invoice reference generation

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo robodoo added the seen 🙂 label Jan 2, 2019

@C3POdoo C3POdoo added the OE label Jan 2, 2019

@robodoo robodoo added the CI 🤖 label Jan 2, 2019

@william-andre

This comment has been minimized.

Copy link
Contributor Author

william-andre commented Jan 17, 2019

TODO need to take care of #29735

william-andre added some commits Feb 11, 2019

[ADD] account: new european invoice reference
Task 1920906
new rf_invoice_number and rf_partner_ref added as possible invoice reference generation
[IMP] l10n_be_invoice_bba: update reference fields
Task 1920906
remove the l10n_be_structured_comm field
'random'/'date' references will not be longer used
new structured_invoice_number and structured_partner_ref reference types

@william-andre william-andre force-pushed the odoo-dev:master-invoice-reference-improvement-wan branch from e02626d to a580094 Feb 11, 2019

@robodoo robodoo removed the CI 🤖 label Feb 11, 2019

@william-andre

This comment has been minimized.

Copy link
Contributor Author

william-andre commented Feb 11, 2019

I think the l10n_be mode for bba_partner_ref is really bad. I mean... max 999 invoices per partner?
Should I remove it or completely rework it? Or keep it as is?

@robodoo robodoo added the CI 🤖 label Feb 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment