[18.0][MIG] datev_export_xml by hbrunn · Pull Request #217 · OCA/l10n-germany

hbrunn · 2026-01-26T15:33:02Z

contains #160

Update datev_export_xml.pot README.rst icon.png Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-germany-14.0/l10n-germany-14.0-datev_export_xml Translate-URL: https://translation.odoo-community.org/projects/l10n-germany-14-0/l10n-germany-14-0-datev_export_xml/

…or tax amount

…dly described exceptions

[UPD] Update datev_export_xml.pot [BOT] post-merge updates Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-germany-15.0/l10n-germany-15.0-datev_export_xml Translate-URL: https://translation.odoo-community.org/projects/l10n-germany-15-0/l10n-germany-15-0-datev_export_xml/

[UPD] Update datev_export_xml.pot [BOT] post-merge updates Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-germany-16.0/l10n-germany-16.0-datev_export_xml Translate-URL: https://translation.odoo-community.org/projects/l10n-germany-16-0/l10n-germany-16-0-datev_export_xml/

…n invoices. Fix some edge cases like fully discounted lines

…le taxes exists

[UPD] Update datev_export_xml.pot [BOT] post-merge updates Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-germany-16.0/l10n-germany-16.0-datev_export_xml Translate-URL: https://translation.odoo-community.org/projects/l10n-germany-16-0/l10n-germany-16-0-datev_export_xml/

TT54915

…rors en export

hbrunn · 2026-01-26T15:33:17Z

/ocabot migration datev_export_xml

mt-software-de

Thx. For the migration.
Code Review only. LGTM.
Could you squash 7e24bb7 into 60c2ff3 ?

hbrunn · 2026-01-28T04:33:16Z

Could you squash 7e24bb7 into 60c2ff3 ?

I'd rather not. This commit was done by another person, so this would destroy attribution. And the feature is simpler to backport if I keep it as separate commit

mt-software-de · 2026-01-28T06:48:18Z

Could you squash 7e24bb7 into 60c2ff3 ?

I'd rather not. This commit was done by another person, so this would destroy attribution. And the feature is simpler to backport if I keep it as separate commit

I haven't found it on any existing branch so i thought it belongs to the migration.
Is it cherry-picked from an existing oca branch, if not could you reword the commit message?
[IMP] datev_export_xml: ...

…f t-elif

…download

…nstead

hbrunn · 2026-01-28T09:38:02Z

changed the commit message

jans23 · 2026-01-29T08:21:27Z

Tested and works well for me.

ivs-cetmix · 2026-02-06T15:29:49Z

Hey @hbrunn thank you for your contribution! Would be great if you could improve the test coverage in a separate commit so we could have it merged.

hbrunn · 2026-02-06T17:15:15Z

like in bec68a9?

ivs-cetmix · 2026-02-06T22:42:26Z

like in bec68a9?

A sort of. We need to have more tests so the Codecov check could pass.

OSevangelist · 2026-03-16T15:40:51Z

@hbrunn @jans23 @ivs-cetmix what is the status here. Can we merge it?

OSevangelist · 2026-03-17T19:34:48Z

@hbrunn @ivs-cetmix i don't know but just increasing the test coverage for sake of improving the computed coverage is pointless. Afaik there is now such policy that would effectively hinder from merging here or am i wrong

ivs-cetmix · 2026-03-18T10:57:53Z

@OSevangelist I'm just following the migration Wiki guidelines https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0
However I agree with you that not having 0.000something test coverage wouldn't heart anyone.
With this said
/ocabot merge nobump

OCA-git-bot · 2026-03-18T10:58:01Z

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-217-by-ivs-cetmix-bump-nobump, awaiting test results.

OCA-git-bot · 2026-03-18T11:05:22Z

Congratulations, your PR was merged at 8f467aa. Thanks a lot for contributing to OCA. ❤️

fkantelberg and others added 23 commits January 26, 2026 05:27

datev_export_xml: Add module to export account.move as DATEV XML

542a673

[MIG] datev_export_xml: Migration to 15.0

ff40b23

[FIX] datev_export_xml: Fix call of sorted for pickings

41fc9d1

[FIX] datev_export_xml: Enforce some limits and use different field f…

5b82694

…or tax amount

[IMP] datev_export_xml: Check invoices before rendering to prevent ba…

e19ffec

…dly described exceptions

[FIX] datev_export_xml: Negate values for refunds (invoice corrections)

7437c20

[FIX] Problem when tax_base_amount is 0 or line names fallback to False

82b3da1

[IMP] : pre-commit stuff

36ec1a2

[MIG] : Migration datev_export_xml to 16.0

12483bb

[IMP] datev_export_xml: Update template invoice_party account

e8d5a61

[IMP] datev_export_xml: Allow previous validation to see all errors o…

d818476

…n invoices. Fix some edge cases like fully discounted lines

[FIX] datev_export_xml: Add general check on invoice e.g. when multip…

2e6ce2d

…le taxes exists

[IMP] datev_export_xml: pre-commit auto fixes

58be409

[MIG] datev_export_xml: Migration to 17.0

91adffd

TT54915

[UPD] Update datev_export_xml.pot

6b48045

[BOT] post-merge updates

e2f8d50

[IMP] datev_export_xml: Add some length constrains to fix possible er…

2384644

…rors en export

[BOT] post-merge updates

bb12055

[IMP] datev_export_xml: pre-commit auto fixes

2a7328a

OCA-git-bot added this to the 18.0 milestone Jan 26, 2026

OCA-git-bot mentioned this pull request Jan 26, 2026

Migration to version 18.0 #167

Open

3 tasks

[MIG] datev_export_xml: Migration to 18.0

60c2ff3

hbrunn force-pushed the 18.0-mig-datev_export_xml branch 2 times, most recently from 518e2e5 to 081553e Compare January 26, 2026 17:01

mt-software-de approved these changes Jan 27, 2026

View reviewed changes

jans23 and others added 11 commits January 28, 2026 10:37

[IMP] datev_export_xml: add security group

f98be19

[FIX] datev_export_xml: Fix duplicate lines because of t-if instead o…

867b76f

…f t-elif

[IMP] datev_export_xml: Add xml declaration

faf692e

[IMP] datev_export_xml: Insert vat_id if available at the partner

af62059

[IMP] datev_export_xml: Improve handling and download of bigger files

c6465c3

[IMP] datev_export_xml: Automatic packaging into smaller zip files

1faceed

[FIX] datev_export_xml: Move counter threshold to packaging

3485c73

[FIX] datev_export_xml: Prevent exception if non-admin user tries to …

05cb053

…download

[FIX] datev_export_xml: Count the number of files for the threshold i…

d634789

…nstead

[IMP] migrate OCA#160 to v18

54e84d0

[IMP] increase test coverage

bec68a9

hbrunn force-pushed the 18.0-mig-datev_export_xml branch from 081553e to bec68a9 Compare January 28, 2026 09:37

jans23 approved these changes Jan 29, 2026

View reviewed changes

OCA-git-bot added the approved label Jan 29, 2026

OCA-git-bot added the bot is merging ⏳ label Mar 18, 2026

OCA-git-bot merged commit 5a3931a into OCA:18.0 Mar 18, 2026
5 of 7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Mar 18, 2026

Uh oh!

Conversation

hbrunn commented Jan 26, 2026

Uh oh!

hbrunn commented Jan 26, 2026

Uh oh!

mt-software-de left a comment

Choose a reason for hiding this comment

Uh oh!

hbrunn commented Jan 28, 2026

Uh oh!

mt-software-de commented Jan 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hbrunn commented Jan 28, 2026

Uh oh!

jans23 commented Jan 29, 2026

Uh oh!

ivs-cetmix commented Feb 6, 2026

Uh oh!

hbrunn commented Feb 6, 2026

Uh oh!

ivs-cetmix commented Feb 6, 2026

Uh oh!

OSevangelist commented Mar 16, 2026

Uh oh!

OSevangelist commented Mar 17, 2026

Uh oh!

ivs-cetmix commented Mar 18, 2026

Uh oh!

OCA-git-bot commented Mar 18, 2026

Uh oh!

Uh oh!

OCA-git-bot commented Mar 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

mt-software-de commented Jan 28, 2026 •

edited

Loading