Skip to content

Conversation

@EmilioPascual
Copy link
Contributor

@EmilioPascual EmilioPascual commented Jun 25, 2025

The company name is displayed on the side of the account when a MIS report is marked as multi-company but has only one effective company. This is not necessary because there is only one effective company and it is not necessary to know which company the account belongs to.

MT-10671 @moduon

Description

Issue: #702

@sbidoul @rafaelbn @yajo please review

…ompany

The company name is displayed on the side of the account when a MIS report is marked as multi-company but has only one effective company. This is not necessary because there is only one effective company and it is not necessary to know which company the account belongs to.

MT-10671
@OCA-git-bot
Copy link
Contributor

Hi @sbidoul,
some modules you are maintaining are being modified, check this out!

Copy link
Member

@yajo yajo left a comment

Choose a reason for hiding this comment

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

Code review

Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

👍🏼 Tested! Check screen shots for a quick review please @sbidoul . Who could merge this one ? 🙏🏼

imagen

After this PR

imagen

Thank you @EmilioPascual ! ❤️ 😄

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@sbidoul
Copy link
Member

sbidoul commented Jul 25, 2025

Looks good, thanks!

Do you plan to forward port to 18?

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-703-by-sbidoul-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 7758f18 into OCA:16.0 Jul 25, 2025
5 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 82d3917. Thanks a lot for contributing to OCA. ❤️

@EmilioPascual
Copy link
Contributor Author

Do you plan to forward port to 18?

Yes, I do

@Xino61122
Copy link

We can see that don't need to migrate to v.18, because it's already done.
Captura desde 2025-07-29 09-02-23

@EmilioPascual EmilioPascual changed the title [IMP] mis_builder: Not display comapny name with only one effective company [IMP] mis_builder: Not display company name with only one effective company Aug 1, 2025
@EmilioPascual EmilioPascual deleted the 16.0-imp-mis_builder-company_name_not_display branch September 17, 2025 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants