-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
[16.0][MIG] l10n_nl_xaf_auditfile_export #371
[16.0][MIG] l10n_nl_xaf_auditfile_export #371
Conversation
and don't output empty fields for enums
via custom widgets
* Align code with l10n_nl_xaf_auditfile_export V8 * Porting to V10 * Update readme * Minor code review * Fix start and end date of periods + updated readme * Fix condition when checking dates * Update Usage in readme according V10 menu names * Replace bank field with bank_id * Make menuitem compatible with Odoo enterprise * Add nl.po file (courtesy of @ploegvde) * Fix menu name in readme * Add empty line in chatter between errors
…e not rounded in python code so could still occur too many decimals error.
…als. [IMP] l10n_nl_xaf_auditfile_export: adding a simple test to include lines from inactive journals.
a4f5d20
to
e3dca6e
Compare
e3dca6e
to
dbe7ec2
Compare
/ocabot migration l10n_nl_xaf_auditfile_export |
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.
👍 LGTM
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.
Functional review.
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 36cd77c. Thanks a lot for contributing to OCA. ❤️ |
No description provided.