Skip to content

[18.0][MIG] datev_export_dtvf: Migration to 18.0#216

Open
hbrunn wants to merge 12 commits intoOCA:18.0from
hbrunn:18.0-mig-datev_export_dtvf
Open

[18.0][MIG] datev_export_dtvf: Migration to 18.0#216
hbrunn wants to merge 12 commits intoOCA:18.0from
hbrunn:18.0-mig-datev_export_dtvf

Conversation

@hbrunn
Copy link
Copy Markdown
Member

@hbrunn hbrunn commented Jan 19, 2026

No description provided.

@hbrunn
Copy link
Copy Markdown
Member Author

hbrunn commented Jan 19, 2026

/ocabot migration datev_export_dtvf

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Jan 19, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 19, 2026
3 tasks
@hbrunn hbrunn force-pushed the 18.0-mig-datev_export_dtvf branch 2 times, most recently from a2152b6 to 875dff4 Compare January 19, 2026 16:32
Copy link
Copy Markdown
Contributor

@jans23 jans23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested the module successfully.

@hbrunn hbrunn force-pushed the 18.0-mig-datev_export_dtvf branch from 875dff4 to 0c752a2 Compare January 26, 2026 16:30
Copy link
Copy Markdown
Contributor

@mt-software-de mt-software-de left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx. This looks great.
I was just not sure where datev_account_code_length was coming from because it was already used and now added as new field.
Until i have seen that in the meantime a migration of datev_export happened, where those fields were removed.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@hbrunn
Copy link
Copy Markdown
Member Author

hbrunn commented Jan 28, 2026

I've added datev_account_code_length in the common module in v15, but as this never made it to v16, v17 and no other module uses those fields anyways, I've rather kept it here

Copy link
Copy Markdown

@hannes386 hannes386 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hbrunn could you please choose a more descriptive name for the module? Like "Datev DTVF Export". Thank you

@hbrunn
Copy link
Copy Markdown
Member Author

hbrunn commented Jan 30, 2026

@hannes386 done

@tv-openbig
Copy link
Copy Markdown
Contributor

@hbrunn Hello, do you have an idea why the runboat build cannot be opened correctly ? Best regards and thanks in advance for your work.

Bildschirmfoto 2026-03-19 um 18 36 55

@hbrunn hbrunn closed this Mar 19, 2026
@hbrunn hbrunn reopened this Mar 19, 2026
@hbrunn
Copy link
Copy Markdown
Member Author

hbrunn commented Mar 19, 2026

/ocabot rebase

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, PR rebased to 18.0.

@OCA-git-bot OCA-git-bot force-pushed the 18.0-mig-datev_export_dtvf branch from 87516ee to fd83545 Compare March 19, 2026 18:29
@hbrunn
Copy link
Copy Markdown
Member Author

hbrunn commented Mar 19, 2026

@tv-openbig builds are garbage collected after a while. you can trigger a rebuild by rebasing

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants