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

[17.0][MIG] pos_analytic_by_config: Migration to 17.0 #652

Open
wants to merge 26 commits into
base: 17.0
Choose a base branch
from
Open

[17.0][MIG] pos_analytic_by_config: Migration to 17.0 #652

wants to merge 26 commits into from

Conversation

peluko00
Copy link

@peluko00 peluko00 commented Apr 30, 2024 •
edited
Loading

adrienpeiffer and others added 25 commits April 30, 2024 09:00
@adrienpeiffer @peluko00
Rename pos_analytic_config to pos_analytic_by_config
6e39c2d
@oca-transbot @peluko00
OCA Transbot updated translations from Transifex
6975e4b
@peluko00
[MIG][10.0] pos_analytic_by_config
a667f5e
@chienandalu @peluko00
[MIG] pos_analytic_by_config: Migration to 12.0
774d467
@chienandalu @peluko00
[FIX] pos_analytic_by_config tests
7096d5c 
Avoid localization issues
@chienandalu @peluko00
[IMP] pos_analytic_by_config: black, isort, prettier
c9036ac
@chienandalu @peluko00
[MIG] pos_analytic_by_config: Migration to 13.0
581961e
@xavier-bouquiaux @peluko00
[14.0][MIG] pos_analytic_by_config
6a06bb8
@Noel000 @peluko00
Translated using Weblate (Catalan)
053277b 
Currently translated at 100.0% (6 of 6 strings)

Translation: account-analytic-14.0/account-analytic-14.0-pos_analytic_by_config
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-14-0/account-analytic-14-0-pos_analytic_by_config/ca/
@douglascstd @peluko00
Translated using Weblate (Portuguese (Brazil))
2960264 
Currently translated at 100.0% (6 of 6 strings)

Translation: account-analytic-14.0/account-analytic-14.0-pos_analytic_by_config
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-14-0/account-analytic-14-0-pos_analytic_by_config/pt_BR/
@RachidAlassir @peluko00
Added translation using Weblate (Arabic)
bebc31a
@RachidAlassir @peluko00
Translated using Weblate (Arabic)
0ae812a 
Currently translated at 100.0% (10 of 10 strings)

Translation: account-analytic-14.0/account-analytic-14.0-pos_analytic_by_config
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-14-0/account-analytic-14-0-pos_analytic_by_config/ar/
@Ricardoalso @peluko00
[MIG] pos_analytic_by_config: Migration to 15.0
7f0c208
@Ivorra78 @peluko00
Translated using Weblate (Spanish)
9e73a00 
Currently translated at 100.0% (6 of 6 strings)

Translation: account-analytic-15.0/account-analytic-15.0-pos_analytic_by_config
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-15-0/account-analytic-15-0-pos_analytic_by_config/es/
@Ricardoalso @peluko00
[15.0][FIX] pos_analytic_by_config: update signature of _get_sale_vals
11124f3
@Ivorra78 @peluko00
Translated using Weblate (Spanish)
4c97210 
Currently translated at 100.0% (7 of 7 strings)

Translation: account-analytic-15.0/account-analytic-15.0-pos_analytic_by_config
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-15-0/account-analytic-15-0-pos_analytic_by_config/es/
@OCA-git-bot @peluko00
[BOT] post-merge updates
1179d03
@chienandalu @peluko00
[MIG] pos_analytic_by_config: Migration to 16.0
223c832 
TT45177
@peluko00
[UPD] Update pos_analytic_by_config.pot
ba89857
@weblate @peluko00
Update translation files
c3c999f 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-16.0/account-analytic-16.0-pos_analytic_by_config
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-pos_analytic_by_config/
@weblate @peluko00
Update translation files
c935643 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-16.0/account-analytic-16.0-pos_analytic_by_config
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-pos_analytic_by_config/
@mymage @peluko00
Added translation using Weblate (Italian)
d01745c
@mymage @peluko00
Translated using Weblate (Italian)
ef5164f 
Currently translated at 100.0% (5 of 5 strings)

Translation: account-analytic-16.0/account-analytic-16.0-pos_analytic_by_config
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-pos_analytic_by_config/it/
@Ivorra78 @peluko00
Translated using Weblate (Spanish)
30a73a6 
Currently translated at 100.0% (5 of 5 strings)

Translation: account-analytic-16.0/account-analytic-16.0-pos_analytic_by_config
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-pos_analytic_by_config/es/
@peluko00
[IMP] pos_analytic_by_config: pre-commit auto fixes
65a69ef
@peluko00 peluko00 mentioned this pull request Apr 30, 2024
Open
15 tasks
@peluko00 peluko00 force-pushed the 17.0-mig-pos_analytic_by_config branch 2 times, most recently from b87e94a to c1bf69c Compare May 2, 2024 11:03
miquelalzanillas
miquelalzanillas approved these changes May 2, 2024
View reviewed changes
Copy link

@miquelalzanillas miquelalzanillas left a comment

Choose a reason for hiding this comment

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

Tested functionally.

image

image

Working as expected.

mpascuall
mpascuall approved these changes May 15, 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.

Functionally working, checked 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). 🤖

@ougc27
Copy link

ougc27 commented Jun 12, 2024

Hi! @peluko00 do you know if this PR would be merged soon? As i can see all the test was passed succesfully...

@peluko00
Copy link
Author

Hi! @peluko00 do you know if this PR would be merged soon? As i can see all the test was passed succesfully...

Hi @ougc27! Is ready to merge, can you test and approve that and maybe @pedrobaeza will merge later

@peluko00 peluko00 force-pushed the 17.0-mig-pos_analytic_by_config branch from c1bf69c to ffe7788 Compare August 28, 2024 06:11
BernatObrador
BernatObrador approved these changes Aug 28, 2024
View reviewed changes
Copy link

@BernatObrador BernatObrador 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

ppyczko
ppyczko approved these changes Aug 29, 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, tested in runboat

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

@miquelalzanillas miquelalzanillas miquelalzanillas approved these changes

@BernatObrador BernatObrador BernatObrador approved these changes

@mpascuall mpascuall mpascuall approved these changes

@ppyczko ppyczko ppyczko approved these changes

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

Successfully merging this pull request may close these issues.