Skip to content

[IMP] product_cost_security: hide standard_price column in product tree views Task#98433#26

Merged
luisg123v merged 1 commit intoVauxoo:18.0from
vauxoo-dev:18.0-task-98433-emtz10
Mar 12, 2026
Merged

[IMP] product_cost_security: hide standard_price column in product tree views Task#98433#26
luisg123v merged 1 commit intoVauxoo:18.0from
vauxoo-dev:18.0-task-98433-emtz10

Conversation

@emtz10
Copy link

@emtz10 emtz10 commented Feb 28, 2026

Adds the 'product_cost_security.group_product_cost' group restriction to the 'standard_price' field in both product.product and product.template tree views.

This ensures that users without the explicit cost permission cannot visualize the cost column when triggering the "Search More" popup from sale orders or when accessing the default list views, strictly enforcing the visibility criteria.

@emtz10 emtz10 force-pushed the 18.0-task-98433-emtz10 branch 2 times, most recently from 7e7ce98 to aa0ce79 Compare February 28, 2026 23:40
…ee views

Adds the 'product_cost_security.group_product_cost' group restriction
to the 'standard_price' field in both product.product and
product.template tree views.

This ensures that users without the explicit cost permission cannot
visualize the cost column when triggering the "Search More" popup from
sale orders or when accessing the default list views, strictly enforcing
the visibility criteria.
@emtz10 emtz10 force-pushed the 18.0-task-98433-emtz10 branch from aa0ce79 to 6301648 Compare February 28, 2026 23:50
@emtz10
Copy link
Author

emtz10 commented Feb 28, 2026

FYI @hugho-ad

@hugho-ad
Copy link
Collaborator

hugho-ad commented Mar 1, 2026

Ci is failing

@emtz10
Copy link
Author

emtz10 commented Mar 1, 2026

Ci is failing

Not related @hugho-ad

Copy link
Collaborator

@hugho-ad hugho-ad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's open PR on the OCA repo pls

@emtz10
Copy link
Author

emtz10 commented Mar 4, 2026

let's open PR on the OCA repo pls

Done OCA#2230

@luisg123v luisg123v merged commit f94e5d8 into Vauxoo:18.0 Mar 12, 2026
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants