Skip to content

[18.0][FIX] web_company_color: show company colors from active company#3274

Merged
OCA-git-bot merged 1 commit intoOCA:18.0from
ForgeFlow:18.0-fix-web_company_color-multi_company_active
Sep 5, 2025
Merged

[18.0][FIX] web_company_color: show company colors from active company#3274
OCA-git-bot merged 1 commit intoOCA:18.0from
ForgeFlow:18.0-fix-web_company_color-multi_company_active

Conversation

@ArnauCForgeFlow
Copy link
Copy Markdown
Contributor

When you have multiple active companies, changing the active company doesn’t update the company colors.

@pedrobaeza pedrobaeza added this to the 18.0 milestone Sep 5, 2025
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Checked on the developer tools that the way to store this in the cookie is with -.

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-3274-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Sep 5, 2025
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Copy Markdown
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-3274-by-pedrobaeza-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@pedrobaeza
Copy link
Copy Markdown
Member

OK, we have the known problem...

Copy link
Copy Markdown
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Tested, it works fine now 👍

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-3274-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2afe8f5 into OCA:18.0 Sep 5, 2025
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@LoisRForgeFlow LoisRForgeFlow deleted the 18.0-fix-web_company_color-multi_company_active branch September 8, 2025 08:52
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.

5 participants