Skip to content

Conversation

@jugurtha-gaci
Copy link

@jugurtha-gaci jugurtha-gaci commented Nov 6, 2025

When the 'Analytic Accounting' feature is not enabled in the Accounting settings, several related menu items are still displayed to the user.
This causes clutter and presents irrelevant options, confusing users.

This PR ensures that all menu items related to analytic accounting are conditionally hidden.

This affects menus under:

  • Accounting -> Analytic Items
  • Reporting -> Analytic Report
  • Configuration -> Analytic Accounting and its submenus

task-5231311


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo
Copy link

robodoo commented Nov 6, 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.

Sammme comment that the others 😄

…g is disabled

When the 'Analytic Accounting' feature is not enabled in the Accounting settings, several related menu items are still displayed to the user.
This causes clutter and presents irrelevant options, confusing users.

This commit ensures that all menu items related to analytic accounting are conditionally hidden.

This affects menus under:
* Accounting -> Analytic Items
* Reporting -> Analytic Report
* Configuration -> Analytic Accounting and its submenus

task-5231311
@jugurtha-gaci jugurtha-gaci force-pushed the 18.0-rd-accounting-onboarding-hide-analytics-menu-jugac branch from 9f29599 to 50d2db1 Compare November 10, 2025 10:10
@jugurtha-gaci jugurtha-gaci changed the title [IMP] account: hide analytic menu items if analytics setting not enabled [IMP] account: hide analytic menu items if analytic accounting setting is disabled Nov 10, 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