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.

Sign up for GitHub

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

Jump to bottom

[18.0][MIG] stock_analytic: Migration to 18.0 #710

Open
wants to merge 61 commits into
base: 18.0
Choose a base branch
from
Open

[18.0][MIG] stock_analytic: Migration to 18.0 #710

wants to merge 61 commits into from

Conversation

peluko00
Copy link
Contributor

@peluko00 peluko00 commented Nov 20, 2024 •
edited
Loading

Hanane ELKHAL and others added 30 commits November 20, 2024 10:27
@peluko00
migration to v7
7096589
@yvanpatry @peluko00
merge with mathieu
60b7aec
@Pierre-Monod @peluko00
update for migration V7
734edb4
@peluko00
add images
7fadecb
@Pierre-Monod @peluko00
update for wilog v7.0
0339c91
@yvanpatry @peluko00
Add contract customer invoice and migrate [IMP] SMS erase print [IMP]…
ede9786 
… Stock analytic XML part is now migrated

Add logo for generic modules
@mathieuvatel @peluko00
 - change the logo of all modules
a86897f 
- migration point, put all the modules to not installable
@fv-clearcorp @peluko00
- Added new module stock_analytic
9190fd9 
This module allows the user to generate analytic information from stock
moves.

- Fixed flake8 and pylint errors.
@andhit-r @peluko00
Adapt original module according OCA Standard
50e40cf 
Remove sale and purchase dependency. Add test

Only assign analytic account if account != valuation acc

Changing field name account_analytic_id

Adjust to OCA latest guidelines. Add some usahe info on README
@AaronHForgeFlow @peluko00
[MIG]stock_analytic to v9
3d51c26
@AaronHForgeFlow @peluko00
[MIG]stock_analytic to v10
d1b3ce1
@jaredkipe @peluko00
11.0 MIG stock_analytic
0eb964f
@jaredkipe @peluko00
Add the ability to pick Analytic account on picking form.
8d07b0b 
Remove remaining encoding hints.
Correct lint in test
Correct flake8 in test
Fix documentation and test_flake8
@rschnapka @peluko00
Translated using Weblate (German)
be6f5e4 
Currently translated at 100.0% (2 of 2 strings)

Translation: account-analytic-11.0/account-analytic-11.0-stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-11-0/account-analytic-11-0-stock_analytic/de/
@AaronHForgeFlow @peluko00
[MIG]stock_analytic to v12
071d09d
@peluko00
[MIG] Updated the tests of the module stock_analytic.
940e081
@marcelsavegnago @peluko00
Translated using Weblate (Portuguese (Brazil))
65aab23 
Currently translated at 100.0% (2 of 2 strings)

Translation: account-analytic-12.0/account-analytic-12.0-stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-12-0/account-analytic-12-0-stock_analytic/pt_BR/
@peluko00
Translated using Weblate (Catalan)
a8072d0 
Currently translated at 100.0% (2 of 2 strings)

Translation: account-analytic-12.0/account-analytic-12.0-stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-12-0/account-analytic-12-0-stock_analytic/ca/
@Jerther @peluko00
[12.0] [FIX] Wrong runbot link in readme
630ced0
@Jerther @peluko00
[12.0] [FIX] stock_analytic: Analytic account field is in wrong page
07497f4 
Fixes OCA#257
@badbole @peluko00
Translated using Weblate (Croatian)
06d4c4a 
Currently translated at 100.0% (3 of 3 strings)

Translation: account-analytic-12.0/account-analytic-12.0-stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-12-0/account-analytic-12-0-stock_analytic/hr/
@max3903 @peluko00
[IMP] stock_analytic: Add analytic account to scrap
37d1d0a
@marylla @peluko00
Translated using Weblate (German)
7330780 
Currently translated at 100.0% (4 of 4 strings)

Translation: account-analytic-12.0/account-analytic-12.0-stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-12-0/account-analytic-12-0-stock_analytic/de/
@alvarorib @peluko00
Translated using Weblate (Portuguese)
e877e12 
Currently translated at 100.0% (4 of 4 strings)

Translation: account-analytic-12.0/account-analytic-12.0-stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-12-0/account-analytic-12-0-stock_analytic/pt/
@ruthbeat @peluko00
Translated using Weblate (Spanish)
e7cc6ac 
Currently translated at 100.0% (4 of 4 strings)

Translation: account-analytic-12.0/account-analytic-12.0-stock_analytic
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-12-0/account-analytic-12-0-stock_analytic/es/
@peluko00
[FIX] Apply analytic account on COGS account when booking customer in…
5f24b9f 
…voice
@jimhoefnagels @peluko00
[IMP] stock_analytic: black, isort
c3a081c
@jimhoefnagels @peluko00
[MIG] stock_analytic: Migration to 13.0
5bfd1b9
@jimhoefnagels @peluko00
stock_analytic 12.0.1.1.2
7ec13ed 
(cherry picked from commit 9255622)
@jimhoefnagels @peluko00
[IMP] stock_analytic: black, isort
5c78b12
aisopuro and others added 3 commits November 20, 2024 10:27
@aisopuro @peluko00
[FIX] stock_analytic: tests fail if mrp also loads
5616d6b
@AungKoKoLin1997 @peluko00
[FIX] stock_analytic
f5185fe 
This commit fixes analytic_distribution not being passed from move to move line when updated after the move line is created.
@OCA-git-bot @peluko00
[BOT] post-merge updates
fce983d
@peluko00 peluko00 mentioned this pull request Nov 20, 2024
Open
10 tasks
ppyczko
ppyczko approved these changes Nov 20, 2024
View reviewed changes
Copy link

@ppyczko ppyczko left a comment

Choose a reason for hiding this comment

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

LGTM!

mpascuall
mpascuall approved these changes Nov 22, 2024
View reviewed changes
Copy link

@mpascuall mpascuall left a comment

Choose a reason for hiding this comment

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

LGTM, tested in runboat

@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). 🤖

@houssine78
Copy link

@pedrobaeza this PR can be merged. thanks !

@dreispt
Copy link
Member

dreispt commented Dec 1, 2024

/ocabot migration stock_analytic
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Dec 1, 2024
@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-710-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Dec 1, 2024
@OCA-git-bot
Merge PR #710 into 18.0
7ea8a43 
Signed-off-by dreispt
@OCA-git-bot
Copy link
Contributor

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

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.

@gbrito
Copy link

gbrito commented Jan 10, 2025

@dreispt is there anything we can do to merge this?

gbrito
gbrito suggested changes Jan 10, 2025
View reviewed changes
Copy link

@gbrito gbrito left a comment

Choose a reason for hiding this comment

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

@peluko00 Can you please push this changes?

stock_analytic/tests/test_stock_picking.py Outdated Show resolved Hide resolved
stock_analytic/tests/test_stock_picking.py Outdated Show resolved Hide resolved
stock_analytic/tests/test_stock_picking.py Outdated Show resolved Hide resolved
@peluko00 peluko00 force-pushed the 18.0-mig-stock_analytic branch from fff262e to dc2ca0a Compare January 13, 2025 08:36
@peluko00
Copy link
Contributor Author

peluko00 commented Jan 13, 2025 •
edited
Loading

@dreispt is there anything we can do to merge this?

Done, thanks!

@dreispt seems it's ready to merge

@gbrito
Copy link

gbrito commented Jan 13, 2025

@peluko00 thanks, good work

@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). 🤖

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

@ppyczko ppyczko ppyczko approved these changes

@mpascuall mpascuall mpascuall approved these changes

@gbrito gbrito gbrito approved these changes

@dreispt dreispt dreispt approved these changes

Assignees
No one assigned
Labels
approved ready to merge
Projects
None yet
Milestone
18.0
Development

Successfully merging this pull request may close these issues.