-
-
Notifications
You must be signed in to change notification settings - Fork 64
[15.0][MIG] datev_export_dtvf: Migration to 15.0 #162
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
Conversation
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
|
@hbrunn Do you know why the runboat build failed ? |
|
it does that sometimes, rebasing helps often /ocabot rebase |
|
Congratulations, PR rebased to 15.0. |
67104cf to
bcf260c
Compare
|
@hbrunn I have checked the export. It looks quite good. But did we miss to integrate the default accounts for the suppress machnaism (setting of "BU-Schlüssel") = 40: https://github.com/hbrunn/l10n-germany/blob/14.0-l10n_de_dtvf-init_hook/datev_export_dtvf/models/account_chart_template.py Thanks for the rebase hint. |
|
@hbrunn I wish to get this PR somehow in the l10n-germany repo. Is it possible to make a functional check on a runboat instance ? |
|
/ocabot rebase |
|
Congratulations, PR rebased to 15.0. |
bcf260c to
e07a12b
Compare
|
@tv-openbig I rebased this again creating a new commit, which is now picked up by runboat correctly |
|
@hbrunn Perfect. Now i can test it properly. This would be great as otherwise the users would have to research the accounts by themselves.
|
|
@hbrunn Technically it works as expected if i configure the "suppress" boolean account: Still i am "bullish" for that module, as it could help a lot in comparision to the odoo EE DATEV export. I do not see this time consuming efforts with that module, especially if we have experienced accountants on the Odoo |
|
/ocabot merge major |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
@hbrunn Ich habe es jetzt erstmal übernommen, auch ohne die Einstellungen für das Boolean Feld (siehe oben). Das kann ja gg. nochmal als Patch kommen. |
|
Congratulations, your PR was merged at a1c5f54. Thanks a lot for contributing to OCA. ❤️ |


This is the migration of #118 which never made it into v14 proper.
I took the liberty to add a little cronjob that does a periodic export.