Skip to content

Conversation

@vicire1
Copy link

@vicire1 vicire1 commented Nov 4, 2025

Removed all groups other than analytic.group_analytic_accounting in the corresponding menuitems so that only the ones whom checked analytics can see those actions

task-5231325

@robodoo
Copy link

robodoo commented Nov 4, 2025

This PR targets the un-managed branch odoo-dev/odoo:18.0-rd-accounting-onboarding-malb, it needs to be retargeted before it can be merged.

@malb-odoo malb-odoo force-pushed the 18.0-rd-accounting-onboarding-malb branch from e0a1783 to 9ae5720 Compare November 4, 2025 08:58
@vicire1 vicire1 force-pushed the 18.0-remove-analytics-stuff-videc branch from 305c873 to 750392b Compare November 4, 2025 09:10
Copy link

@malb-odoo malb-odoo left a comment

Choose a reason for hiding this comment

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

Change is good, nice work !! Let's change the commit message to make this perfect 😄

@vicire1 vicire1 force-pushed the 18.0-remove-analytics-stuff-videc branch from 750392b to c74dc74 Compare November 5, 2025 10:38
@vicire1 vicire1 changed the title [IMP] Account: Hide analytics stuff when analytics is unchecked [IMP] account: hide analytics stuff when analytics is unchecked Nov 5, 2025
@vicire1 vicire1 force-pushed the 18.0-remove-analytics-stuff-videc branch from c74dc74 to e556bc8 Compare November 5, 2025 14:47
When analytic is not checked in the accountings parameters it still shows some point of the menu that are available only if you enabled analytics so I made the view more restriccted to see them only when you check the parameter

task-5231325
@vicire1 vicire1 force-pushed the 18.0-remove-analytics-stuff-videc branch from e556bc8 to e514ccb Compare November 5, 2025 16:53
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.

4 participants