[11.0][ENH] sql view - allow group operator #351
Conversation
Replaces OCA#344 Adds the possibility, for float and integer columns, to apply a group operator (average, min, max).
|
@CasVissers Do you know what we need to do to get this merged? |
|
@legalsylvain Think this one is also ready to merge? |
|
This PR has the |
|
/ocabot merge |
|
This PR looks fantastic, let's merge it! |
|
@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 11.0-ocabot-merge-pr-351-by-legalsylvain-bump-no. 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. |
|
@legalsylvain :( I don't understand the error - my expertise is not that great. It appears to be a test unrelated to my changes !? |
|
Hi @richard-willdooit : I have no idea. |
|
/ocabot merge |
|
What a great day to merge this nice PR. Let's do it! |
|
@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 11.0-ocabot-merge-pr-351-by-dreispt-bump-no. 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. |
|
The |
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 7a09e28. Thanks a lot for contributing to OCA. ❤️ |
Replaces #344
Adds the possibility, for float and integer columns, to
apply a group operator (average, min, max).