-
-
Notifications
You must be signed in to change notification settings - Fork 692
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.
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
[14.0] ADD account_receipt_journal: Define and use journals dedicated to receipts #1260
Conversation
https://github.com/OCA/account-invoicing/actions/runs/3219774473/jobs/5265595874 @primes2h does this error say anything to you? |
03f6c12
to
6ebc1af
Compare
Mmhh... It's due to this, committed 7 days ago: I'll try to fix |
6ebc1af
to
a9b7858
Compare
|
651cd5d
to
72ff41e
Compare
72ff41e
to
4a14817
Compare
Aside from the warning, tested it on a local installation (in the context of a migration from 12.0 and the previous l10n_it module) and it seems to be working great! I used these queries in a openupgrade.logged_query(
env.cr,
"""UPDATE account_move am
SET type = 'out_receipt'
where am.id in (SELECT am2.id
FROM account_move am2
JOIN account_invoice ai on am2.old_invoice_id = ai.id
WHERE ai.type = 'out_invoice'
AND ai.corrispettivo = true)""",
)
openupgrade.logged_query(
env.cr,
"""UPDATE account_move am
SET type = 'in_receipt'
where am.id in (SELECT am2.id
FROM account_move am2
JOIN account_invoice ai on am2.old_invoice_id = ai.id
WHERE ai.type = 'in_invoice'
AND ai.corrispettivo = true)""",
) maybe they can be used in the migrations of this module or directly in openupgrade? |
This is included in #1267 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @eLBati ! 👍
Great thing!
/ocabot rebase |
[imp] not working view [imp] not working defaults [IMP] working draft [IMP] semplified view [ADD] translations [FIX] partner type [fix] defaults [IMP] configuration wizard [imp] translation [imp] translation [IMP] description [FIX] copyright [DEL] every module, as they are developed for 6.1. As soon as we port a module to 7.0, before 7.0 was released, we'll put it here
moves all modules into a __unported__ directory
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_corrispettivi Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_corrispettivi/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_corrispettivi Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_corrispettivi/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_corrispettivi Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_corrispettivi/
…eipts Ported from l10n_it_corrispettivi
Congratulations, PR rebased to 14.0. |
224955a
to
72c42fe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ocabot merge nobump
On my way to merge this fine PR! |
It looks like something changed on |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at ba7f3fd. Thanks a lot for contributing to OCA. ❤️ |
No description provided.