-
-
Notifications
You must be signed in to change notification settings - Fork 52
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
[13.0][MIG] datev_export_xml : Migration to version 13.0 #137
Conversation
f5d9138
to
979f4a3
Compare
979f4a3
to
f2a1272
Compare
f844022
to
77df195
Compare
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. |
77df195
to
2ef80c2
Compare
@@ -81,6 +79,7 @@ def setUp(self): | |||
self.refund_date = self.today - timedelta(days=55) | |||
self.start_date = self.today - timedelta(days=34) | |||
self.end_date = self.today - timedelta(days=32) | |||
self.InvoiceObj.search([("state", "=", "posted")]).button_draft() |
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.
Why is it necessary?
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.
This is necessary, as there is a restriction in account move that you cannot delete an item linked to a posted entry.
13f67b1
to
5c50214
Compare
5c50214
to
c521527
Compare
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.
thanks!
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 341f9a5. Thanks a lot for contributing to OCA. ❤️ |
-Standard migration to v13.
ValueError: empty namespace prefix is not supported in ElementPath
TT46193
@pedrobaeza