Skip to content

Conversation

@yagho-odoo
Copy link

No description provided.

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

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.

Solution chosen is the right one ! Good job on this 😄 Let's elaborate the commit message a bit more and then it will be good to go 😄

@yagho-odoo yagho-odoo force-pushed the 18.0-accountingOnboarding-bugpovement-yagho branch from 323b722 to c1ddbc6 Compare November 5, 2025 10:30
@yagho-odoo yagho-odoo changed the title [IMP] accounting: adjusted analytic items menu item groups [IMP] account: adjusted the groups in the menu item so it only appears when analytics is checked in the configuration settings Nov 5, 2025
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.

Same commit message doesn't follow the guideline 😄

…tting is off

When analytics were off in the accounting configuration settings, analytic items were still showing
in the accounting menu, now it only appears if analytics are checked
task id: 5231297
@yagho-odoo yagho-odoo force-pushed the 18.0-accountingOnboarding-bugpovement-yagho branch from c1ddbc6 to 9f0ca55 Compare November 5, 2025 16:37
@yagho-odoo yagho-odoo changed the title [IMP] account: adjusted the groups in the menu item so it only appears when analytics is checked in the configuration settings [IMP] account: remove analytic items from the menu when associated setting is off Nov 5, 2025
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