[11.0][FIX] l10n_nl_tax_statement: add past invoices in report #220
Conversation
astirpe
force-pushed
the
11_fix_l10n_nl_tax_statement
branch
from
11cd64a
to
05ec37e
Compare
astirpe
force-pushed
the
11_fix_l10n_nl_tax_statement
branch
2 times, most recently
from
3214fd6
to
a63bb36
Compare
astirpe
force-pushed
the
11_fix_l10n_nl_tax_statement
branch
from
a63bb36
to
1c38c60
Compare
astirpe
force-pushed
the
11_fix_l10n_nl_tax_statement
branch
2 times, most recently
from
ef93d8c
to
02f7bbe
Compare
View past invoices moves when country is not NL [FIX] Sort past invoices by date [FIX] Domain condition Update roadmap Fix _compute_is_invoice_basis()
astirpe
force-pushed
the
11_fix_l10n_nl_tax_statement
branch
from
02f7bbe
to
b57981e
Compare
|
/ocabot merge minor |
|
What a great day to merge this nice PR. Let's do it! |
|
@NL66278 your merge command was aborted due to failed check(s), which you can inspect on this commit of 11.0-ocabot-merge-pr-220-by-NL66278-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
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. |
github-actions
bot
added
the
stale
|
/ocabot merge minor |
|
On my way to merge this fine PR! |
|
@NL66278 your merge command was aborted due to failed check(s), which you can inspect on this commit of 11.0-ocabot-merge-pr-220-by-NL66278-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
github-actions
bot
removed
the
stale
|
/ocabot merge minor |
|
On my way to merge this fine PR! |
|
@NL66278 your merge command was aborted due to failed check(s), which you can inspect on this commit of 11.0-ocabot-merge-pr-220-by-NL66278-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
Most tests succeed, but when excluding the postcode api module, the test fails on a kvk authorization problem. Travis truly keeps surprising me... |
|
Needs merge of this and then rebase: #392 |
|
@astirpe I intend to close all the 11.0 PR's, as it would be quite a lot of work to get them to pass all tests, because of all the problems. Do you agree? The alternative would be to merge them by using the github button, instead of through the OCA bot. Which would be fine by me as well. |
|
@NL66278 no preference by my side, both are fine for me 👍 thanks! |
Fixes #219