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

[MIG] account_invoice_pricelist: Migration to 15.0 #1111

Merged
merged 44 commits into from
Dec 21, 2022
Merged

[MIG] account_invoice_pricelist: Migration to 15.0 #1111

merged 44 commits into from
Dec 21, 2022

Conversation

nicomacr
Copy link
Contributor

No description provided.

legalsylvain and others added 30 commits March 10, 2022 15:45
@legalsylvain @nicomacr
new module account_invoice_pricelist;
b76b4b7
@oca-transbot @nicomacr
OCA Transbot updated translations from Transifex
ca66f08
@hbrunn @nicomacr
use pricelist when calculating prices
522bdbd
@sergio-teruel @nicomacr
account_invoice_pricelist: Migration to v9
921f4d0
@oca-transbot @nicomacr
OCA Transbot updated translations from Transifex
643c2f4 
OCA Transbot updated translations from Transifex

OCA Transbot updated translations from Transifex
@rvalyi @nicomacr
account_invoice_pricelist: Migration to 10.0
147aa9b
@oca-transbot @nicomacr
OCA Transbot updated translations from Transifex
4c92e76
@amcor @nicomacr
account_invoice_pricelist: Migration to 11.0
349671d
@oca-travis @nicomacr
Update account_invoice_pricelist.pot
4ed65b1
@OCA-git-bot @nicomacr
README.rst
23bc337
@rschnapka @nicomacr
Translated using Weblate (German)
b66cd94 
Currently translated at 100,0% (5 of 5 strings)

Translation: account-invoicing-11.0/account-invoicing-11.0-account_invoice_pricelist
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-11-0/account-invoicing-11-0-account_invoice_pricelist/de/
@OCA-git-bot @nicomacr
README.rst
d9a02b1
@Nikul-Chaudhary @nicomacr
[MIG] account_invoice_pricelist v11 to v12
1614f86
@OCA-git-bot @nicomacr
[UPD] README.rst
5d6a493
@oca-travis @nicomacr
[UPD] Update account_invoice_pricelist.pot
a367a1d
@oca-transbot @nicomacr
Update translation files
1f2fc83 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_pricelist
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_pricelist/
@OCA-git-bot @nicomacr
[ADD] icon.png
56a2920
@bealdav @nicomacr
FIX acc_invoice_pricelist: tax include in multicompany
9d26492
@OCA-git-bot @nicomacr
account_invoice_pricelist 12.0.1.0.1
7575990
@bealdav @nicomacr
FIX acc_invoice_pricelist: recompute tax after update pricelist
24cfff0
@OCA-git-bot @nicomacr
account_invoice_pricelist 12.0.1.0.2
c466732
@am-technix @nicomacr
Translated using Weblate (French)
22549be 
Currently translated at 100.0% (5 of 5 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_pricelist
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_pricelist/fr/
@OCA-git-bot @nicomacr
[UPD] README.rst
b6dc44d
@legalsylvain @nicomacr
[FIX] bad display on invoices form
ddf4691
@OCA-git-bot @nicomacr
account_invoice_pricelist 12.0.1.0.3
11c1561
@nicomacr
Translated using Weblate (Portuguese (Brazil))
7c20114 
Currently translated at 100.0% (5 of 5 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_pricelist
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_pricelist/pt_BR/
@nicomacr
Added translation using Weblate (Spanish (Chile))
53eedcd
@nicomacr
Translated using Weblate (Spanish (Chile))
3fadb5a 
Currently translated at 100.0% (5 of 5 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_pricelist
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_pricelist/es_CL/
@badbole @nicomacr
Translated using Weblate (Croatian)
0df1367 
Currently translated at 100.0% (5 of 5 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_pricelist
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_pricelist/hr/
@manuelregidor @nicomacr
[MIG] account_invoice_pricelist: black, isort, prettier
8084779
@nicomacr nicomacr mentioned this pull request Mar 30, 2022
Merged
@nicomacr nicomacr force-pushed the 15.0-t-26333-nmr2 branch from 3e1c394 to 1f6b375 Compare April 6, 2022 12:02
@nicomacr nicomacr mentioned this pull request Apr 6, 2022
Open
56 tasks
@rafaelbn
Copy link
Member

rafaelbn commented Apr 6, 2022

/ocabot migration account_invoice_pricelist

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Apr 6, 2022
@bhaveshselarka
Copy link

Hello @nicomacr thanks for the migration, Is this ready for merge and production use ??

@marylla
Copy link
Contributor

marylla commented Jun 30, 2022

Hi @nicomacr Could you please rebase? Currently I cannot test it on runboat...

@jabelchi
Copy link
Contributor

jabelchi commented Aug 8, 2022

It's not possible to test it in rumboat.
Am I doing something wrong?

@marylla
Copy link
Contributor

marylla commented Aug 29, 2022

@nicomacr Are you still there to fix this?

@nicomacr
Copy link
Contributor Author

@marylla yes i made the rebase now, but nothing changes in the runboat.

@marylla
Copy link
Contributor

marylla commented Aug 30, 2022

@nicomacr I also cannot see any changes in this communication. Are you sure you rebased and updated this PR?

fkantelberg
fkantelberg suggested changes Nov 29, 2022
View reviewed changes
Copy link
Member

@fkantelberg fkantelberg left a comment

Choose a reason for hiding this comment

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

I tested it locally and it works fine. Just a small comment regarding the returns in the onchange

account_invoice_pricelist/models/account_move.py Outdated Show resolved Hide resolved
navarrorico
navarrorico approved these changes Nov 30, 2022
View reviewed changes
Copy link

@navarrorico navarrorico left a comment

Choose a reason for hiding this comment

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

Functional review 👍

jabelchi
jabelchi approved these changes Dec 1, 2022
View reviewed changes
Copy link
Contributor

@jabelchi jabelchi left a comment

Choose a reason for hiding this comment

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

Functional test on runboat, OK.

@marylla
Copy link
Contributor

marylla commented Dec 20, 2022

I tested it locally and it works fine. Just a small comment regarding the returns in the onchange

@nicomacr Could you check this please? :)

@grindtildeath
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-1111-by-grindtildeath-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3ea0d9b into OCA:15.0 Dec 21, 2022
@OCA-git-bot
Copy link
Contributor

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

@nicomacr nicomacr deleted the 15.0-t-26333-nmr2 branch December 21, 2022 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fkantelberg fkantelberg fkantelberg requested changes

@jabelchi jabelchi jabelchi approved these changes

@navarrorico navarrorico navarrorico approved these changes

@grindtildeath grindtildeath grindtildeath approved these changes

Assignees
No one assigned
Labels
merged 🎉 needs review
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.