-
-
Notifications
You must be signed in to change notification settings - Fork 239
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
[14.0][IMP] sale_operating_unit: allowed to hide operating_unit_id in list views #657
[14.0][IMP] sale_operating_unit: allowed to hide operating_unit_id in list views #657
Conversation
Hello @AaronHForgeFlow Could you please check this small adjustment? regards |
put optional="show" instead That way there is no changes for current users. |
83969e2
to
793e682
Compare
793e682
to
f687ead
Compare
It's done. regards |
@alexeirivera87 Thanks. You can add yourself in the the contributors.rst file but please do not change Readme.rst file. The oca bot will do that. |
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.
LGTM
@AaronHForgeFlow |
@alexeirivera87 no, it is not that important |
I don't mind to merge it just waiting for some other users to approve |
Hello @MiquelRForgeFlow I saw you approved other PR in this repository. Could you please check this one? Thanks in advance. |
Hello @AaronHForgeFlow Can this PR be merged? regards |
ok, let's do it, it is a minor change. The other needed approval is implicit in the other comment. /ocabot merge minor |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 021d5bb. Thanks a lot for contributing to OCA. ❤️ |
added optional="hide"