[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
ArnauCForgeFlow
left a comment
There was a problem hiding this comment.
LGTM! Code and Functional Review, thanks for the improvement!
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. |
Improve the feature to work with any number of Group By filters applied simultaneously.
8d3b479 to
4515f78
Compare
Improve the feature to work with any number of Group By filters applied simultaneously.