Skip to content

[19.0][MIG] analytic_brand: Migration to 19.0#273

Open
bizzappdev wants to merge 34 commits into
OCA:19.0from
BizzAppDev-Systems:19.0-mig-analytic_brand-BAD
Open

[19.0][MIG] analytic_brand: Migration to 19.0#273
bizzappdev wants to merge 34 commits into
OCA:19.0from
BizzAppDev-Systems:19.0-mig-analytic_brand-BAD

Conversation

@bizzappdev
Copy link
Copy Markdown

@bizzappdev bizzappdev commented Oct 8, 2025 •
edited
Loading

No description provided.

@bizzappdev bizzappdev marked this pull request as ready for review October 8, 2025 14:29
@marielejeune
Copy link
Copy Markdown
Contributor

marielejeune commented Dec 19, 2025

Hi,
Please note that I propose some refactoring in v18 (that could then be forward ported to v19). To me it is better to make use of standard account analytic distribution models instead of defining the analytic distribution on the brand and propagating it everywhere.
Check my PR here: #286

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 19, 2026

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 19, 2026
sbejaoui and others added 26 commits April 24, 2026 16:06
@sbejaoui @bizzappdev
[ADD] - Add analytic brand module
ce4eff7
@sbejaoui @bizzappdev
[ADD] - Add analytic_account_id field
e7bae0c
@sbejaoui @bizzappdev
[IMP] - set the invoice line analytic account at brand change
0284077
@oca-travis @bizzappdev
[UPD] Update analytic_brand.pot
b77dbf1 
[UPD] README.rst
@OCA-git-bot @bizzappdev
analytic_brand 12.0.1.0.1
e07dc6a
@sbejaoui @bizzappdev
[MIG] - migration analytic_brand 13.0
5025d87
@oca-travis @bizzappdev
[UPD] Update analytic_brand.pot
087fddb 
[UPD] README.rst

[FIX] Fixed travis.
@AmmarOfficewalaSerpentCS @bizzappdev
[MIG][14.0] Migarted analytic_brand module.
e4e0250 
[UPD] Update analytic_brand.pot
@sbejaoui @bizzappdev
[15.0][MIG] - analytic_brand
4501363
@bizzappdev
[UPD] Update analytic_brand.pot
a668717
@OCA-git-bot @bizzappdev
[UPD] README.rst
0c96617
@BT-tkarpinski @bizzappdev
[FIX] brand view in analytic_brand
1166068
@OCA-git-bot @bizzappdev
analytic_brand 15.0.1.0.1
a37c225
@OCA-git-bot @bizzappdev
[UPD] README.rst
17df973
@Ivorra78 @bizzappdev
Added translation using Weblate (Spanish)
c57181d
@Ivorra78 @bizzappdev
Translated using Weblate (Spanish)
9f871dc 
Currently translated at 100.0% (2 of 2 strings)

Translation: brand-15.0/brand-15.0-analytic_brand
Translate-URL: https://translation.odoo-community.org/projects/brand-15-0/brand-15-0-analytic_brand/es/
@JasminSForgeFlow @bizzappdev
[IMP] analytic_brand: black, isort, prettier
49ee721
@JasminSForgeFlow @bizzappdev
[MIG] analytic_brand: Migration to 17.0
e25dab1
@bizzappdev
[UPD] Update analytic_brand.pot
dfdcac7
@OCA-git-bot @bizzappdev
[BOT] post-merge updates
db65069
@mymage @bizzappdev
Added translation using Weblate (Italian)
24836a2
@mymage @bizzappdev
Translated using Weblate (Italian)
c6a85e6 
Currently translated at 100.0% (2 of 2 strings)

Translation: brand-17.0/brand-17.0-analytic_brand
Translate-URL: https://translation.odoo-community.org/projects/brand-17-0/brand-17-0-analytic_brand/it/
@BhaveshHeliconia @bizzappdev
[MIG] analytic_brand: Migration to 18.0
7459626
@marielejeune @bizzappdev
[CHG] analytic_brand: replace analytic account by analytic distribution
0c14572
@bizzappdev
[UPD] Update analytic_brand.pot
ad4008d
@OCA-git-bot @bizzappdev
[BOT] post-merge updates
189ad32
marielejeune and others added 8 commits April 24, 2026 16:06
@marielejeune @bizzappdev
[FIX] analytic_brand: Run tests in post-install:
97f0808 
There is a common issue in v18 when creating a new res.partner in tests
(which is the case here as we create a res.brand).
There is a required field autopost_bills in account addon.
As this addon doesn't depend on account, if we install this addon
on a DB where account is also installed, tests will fail because
the autopost_bills column is created into postgres, with the not null constraint,
but the default value won't be set by Odoo.

Running these tests in post-install ensure Odoo will set the default value
as specified in the account addon.
@bizzappdev
[UPD] Update analytic_brand.pot
30bc55f
@OCA-git-bot @bizzappdev
[BOT] post-merge updates
e7b2eba
@weblate @bizzappdev
Update translation files
dd07446 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: brand-18.0/brand-18.0-analytic_brand
Translate-URL: https://translation.odoo-community.org/projects/brand-18-0/brand-18-0-analytic_brand/
@mymage @bizzappdev
Translated using Weblate (Italian)
3440778 
Currently translated at 100.0% (4 of 4 strings)

Translation: brand-18.0/brand-18.0-analytic_brand
Translate-URL: https://translation.odoo-community.org/projects/brand-18-0/brand-18-0-analytic_brand/it/
@MRomeera @bizzappdev
Translated using Weblate (Spanish)
520cfa0 
Currently translated at 100.0% (4 of 4 strings)

Translation: brand-18.0/brand-18.0-analytic_brand
Translate-URL: https://translation.odoo-community.org/projects/brand-18-0/brand-18-0-analytic_brand/es/
@bizzappdev
[IMP] analytic_brand: pre-commit auto fixes
d5357e5
@bizzappdev
[MIG] analytic_brand: Migration to 19.0
0738d7f
@bizzappdev bizzappdev force-pushed the 19.0-mig-analytic_brand-BAD branch from 9e96993 to 0738d7f Compare April 24, 2026 10:36
@bizzappdev
Copy link
Copy Markdown
Author

bizzappdev commented Apr 24, 2026

Rebase done as dependency PR is merged.

@github-actions github-actions Bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 26, 2026
@BhaveshHeliconia
Copy link
Copy Markdown
Contributor

BhaveshHeliconia commented May 18, 2026

Hi, Please note that I propose some refactoring in v18 (that could then be forward ported to v19). To me it is better to make use of standard account analytic distribution models instead of defining the analytic distribution on the brand and propagating it everywhere. Check my PR here: #286

@bizzappdev could you please take a look at this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

mod:analytic_brand Module analytic_brand series:19.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.