[18.0][IMP] web_excel_export_dynamic_expand: support several group by filters#3576
[18.0][IMP] web_excel_export_dynamic_expand: support several group by filters#3576LoisRForgeFlow wants to merge 1 commit into
Conversation
Improve the feature to work with any number of Group By filters applied simultaneously.
OCA-git-bot
added
series:18.0
mod:web_excel_export_dynamic_expand
ArnauCForgeFlow
left a comment
ArnauCForgeFlow
left a comment
There was a problem hiding this comment.
LGTM! Code and Functional Review, thanks for the improvement!
etobella
left a comment
etobella
left a comment
There was a problem hiding this comment.
Tested in runboat
It would have been awesome to have a more detailed explanation to find the change easily 😉
|
@OCA/web-maintainers could you merge? |
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
@etobella your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-3576-by-etobella-bump-nobump. 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. |
5 participants
Improve the feature to work with any number of Group By filters applied simultaneously.