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_sale: Migration to 15.0 #1092

Merged
merged 27 commits into from
Jan 4, 2023
Merged

[MIG] account_invoice_pricelist_sale: Migration to 15.0 #1092

merged 27 commits into from
Jan 4, 2023

Conversation

nicomacr
Copy link
Contributor

@nicomacr nicomacr commented Feb 2, 2022

No description provided.

@ajaniszewska-dev
Copy link
Contributor

@nicomacr tests are failing, pls check it.

@nicomacr nicomacr force-pushed the 15.0-t-26333-nmr branch 2 times, most recently from 648188f to 1a0320d Compare February 8, 2022 19:08
@nicomacr
Copy link
Contributor Author

waiting this PR #1111

@marylla
Copy link
Contributor

marylla commented Jun 30, 2022

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

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.

Please rebase on the current 15.0 branch. This should fix the CI. Otherwise it works locally.

@navarrorico
Copy link

Works fine for me, functional review 👍

@grindtildeath
Copy link
Contributor

/ocabot migration account_invoice_pricelist_sale

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Dec 21, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 21, 2022
Open
56 tasks
@grindtildeath
Copy link
Contributor

@nicomacr Can you please rebase?

NL66278 and others added 15 commits December 21, 2022 14:51
@NL66278 @nicomacr
[ENH] Make sure pricelist on invoice is used.
ee5e0ae
@oca-transbot @nicomacr
OCA Transbot updated translations from Transifex
62013b7 
OCA Transbot updated translations from Transifex
@rvalyi @nicomacr
v10 migr of account_invoice_pricelist_sale
6711af0
@rvalyi @nicomacr
v10 migr: change version number
8a8235a
@oca-transbot @nicomacr
OCA Transbot updated translations from Transifex
e02f7a1
@oca-travis @nicomacr
[UPD] Update account_invoice_pricelist_sale.pot
b3ca40f
@OCA-git-bot @nicomacr
[ADD] icon.png
ac8c32b
@legalsylvain @nicomacr
[PORT] account_invoice_pricelist_sale in 12.0
aeb7bf7
@oca-travis @nicomacr
[UPD] Update account_invoice_pricelist_sale.pot
e550d47
@OCA-git-bot @nicomacr
[UPD] README.rst
de47406
@oca-transbot @nicomacr
Update translation files
892792b 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_pricelist_sale
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_pricelist_sale/
@oca-transbot @nicomacr
Update translation files
da82321 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_pricelist_sale
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_pricelist_sale/
@nicomacr
Translated using Weblate (Spanish)
23479f2 
Currently translated at 100.0% (1 of 1 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_pricelist_sale
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_pricelist_sale/es/
@marylla @nicomacr
Translated using Weblate (German)
c57c4c0 
Currently translated at 100.0% (1 of 1 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_pricelist_sale
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_pricelist_sale/de/
@manuelregidor @nicomacr
[IMP] account_invoice_pricelist_sale: black, isort, prettier
e17c680
manuelregidor and others added 11 commits December 21, 2022 14:51
@manuelregidor @nicomacr
[MIG]account_invoice_pricelist_sale: Migration to 13.0
5e4ae8b
@oca-travis @nicomacr
[UPD] Update account_invoice_pricelist_sale.pot
0b669e7
@OCA-git-bot @nicomacr
[UPD] README.rst
7b8e54d
@oca-transbot @nicomacr
Update translation files
9f2c519 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-invoicing-13.0/account-invoicing-13.0-account_invoice_pricelist_sale
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-13-0/account-invoicing-13-0-account_invoice_pricelist_sale/
@kittiu @nicomacr
[13.0][FIX] account_invoice_pricelist_sale, fix test
c5c97d0
@OCA-git-bot @nicomacr
account_invoice_pricelist_sale 13.0.1.0.1
1a6c267
@oca-transbot @nicomacr
Update translation files
07483b6 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-invoicing-13.0/account-invoicing-13.0-account_invoice_pricelist_sale
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-13-0/account-invoicing-13-0-account_invoice_pricelist_sale/
@anasuarez1 @nicomacr
Translated using Weblate (Spanish)
bdb7ac2 
Currently translated at 100.0% (1 of 1 strings)

Translation: account-invoicing-13.0/account-invoicing-13.0-account_invoice_pricelist_sale
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-13-0/account-invoicing-13-0-account_invoice_pricelist_sale/es/
@vincent-hatakeyama @nicomacr
Translated using Weblate (French)
131e012 
Currently translated at 100.0% (1 of 1 strings)

Translation: account-invoicing-13.0/account-invoicing-13.0-account_invoice_pricelist_sale
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-13-0/account-invoicing-13-0-account_invoice_pricelist_sale/fr/
@nicomacr
[IMP] account_invoice_pricelist_sale: black, isort, prettier
c350298
@nicomacr
[MIG] account_invoice_pricelist_sale: Migration to 15.0
f922dcc
@nicomacr
Copy link
Contributor Author

@nicomacr Can you please rebase?

Done!

marylla
marylla approved these changes Dec 22, 2022
View reviewed changes
Copy link
Contributor

@marylla marylla left a comment

Choose a reason for hiding this comment

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

I made a functional review in runboat.
It works fine. The pricelist is passed from sale order to invoice.

jabelchi
jabelchi approved these changes Dec 27, 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 is working for me.
Tested on runboat

@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-1092-by-grindtildeath-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b96c981 into OCA:15.0 Jan 4, 2023
@OCA-git-bot
Copy link
Contributor

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

@nicomacr nicomacr deleted the 15.0-t-26333-nmr branch January 4, 2023 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajaniszewska-dev ajaniszewska-dev ajaniszewska-dev left review comments

@grindtildeath grindtildeath grindtildeath approved these changes

@fkantelberg fkantelberg fkantelberg requested changes

@jabelchi jabelchi jabelchi approved these changes

@hildickethan hildickethan hildickethan approved these changes

@marylla marylla marylla approved these changes

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

Successfully merging this pull request may close these issues.