[14.0][IMP] sale_operating_unit: allowed to hide operating_unit_id in list views

OCA / operating-unit

Manage Operating Units in Odoo

GNU Affero General Public License v3.0

69 stars 235 forks source link

Open alexeirivera87 opened 3 months ago

alexeirivera87 commented 3 months ago

added optional="hide"

alexeirivera87 commented 3 months ago

Hello @AaronHForgeFlow

Could you please check this small adjustment?

regards

AaronHForgeFlow commented 3 months ago

put optional="show" instead

That way there is no changes for current users.

alexeirivera87 commented 3 months ago

@AaronHForgeFlow

It's done.

regards

AaronHForgeFlow commented 3 months ago

@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.

alexeirivera87 commented 2 months ago

@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.

@AaronHForgeFlow Ah, copied. I did not know that. Should I revert the change in the readme file? In order to get this PR approved/merged?

AaronHForgeFlow commented 2 months ago

@alexeirivera87 no, it is not that important

AaronHForgeFlow commented 2 months ago

I don't mind to merge it just waiting for some other users to approve

alexeirivera87 commented 6 days ago

Hello @MiquelRForgeFlow

I saw you approved other PR in this repository. Could you please check this one? Thanks in advance.