Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMP] base, account: remove l10n install banner from company #159245

Open
wants to merge 1 commit into
base: saas-17.1
Choose a base branch
from

Conversation

hamo-odoo
Copy link
Contributor

Before this commit:
Whenever accounting or HR module was installed, A blue banner was visible on company form if the
available localisation were not installed based on company country

After this commit:
We don't show install l10n modules banner instead only if accounting is installed and Chart of Account is not installed for the company then config banner is visible on company which will redirect the user to Account Setting to set up the Chart of Account

task-3778290


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

@robodoo
Copy link
Contributor

robodoo commented Mar 26, 2024

@C3POdoo C3POdoo requested review from a team, hupo-odoo, rco-odoo and sleepy-monax and removed request for a team March 26, 2024 08:50
@C3POdoo C3POdoo added the RD research & development, internal work label Mar 26, 2024
@hamo-odoo hamo-odoo force-pushed the saas-17.1-rem-auto-install-hamo branch from 62355e5 to 337a713 Compare March 26, 2024 09:06
Before this commit:
Whenever `accounting` or `HR` module was installed,
A blue banner was visible on company form if the
available localisation were not installed based on
company country

After this commit:
We don't show install l10n modules banner
instead only if `accounting` is installed
and Chart of Account is not installed for
the company then config banner is visible
on company which will redirect the user
to Account Setting to set up the Chart
of Account

task-3778290
@hamo-odoo hamo-odoo force-pushed the saas-17.1-rem-auto-install-hamo branch from 337a713 to a2a3e1b Compare March 26, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants