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] account_invoice_merge: Migration to 17.0 #1632

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

[17.0][MIG] account_invoice_merge: Migration to 17.0 #1632

wants to merge 75 commits into from

Conversation

moitabenfdz
Copy link

No description provided.

[email protected] and others added 30 commits December 21, 2023 13:44
@moitabenfdz
[ADD] several new modules double confirmation, web wysiwyg etc.
2c583c8 
(../elico-6.0/ rev 42)
@moitabenfdz
[FIX] correct naming bug
431d02e 
(../elico-6.0/ rev 44)
@moitabenfdz
account_invoice_merge: migrate module in V7. Add french translation.
e17e2cb
@moitabenfdz
account_invoice_merge: in copyright replace Erico-Corp by Elico Corp.
8ad0f16
@moitabenfdz
account_invoice_merge : replace osv by orm
a7b1a8a
@moitabenfdz
account_invoice_merge : replace label from invoice_merge_view.xml wit…
a9aa88b 
…h html
@moitabenfdz
account_invoice_merge : replace "todo_ids +=" by "todo_ids" and move …
7860408 
…it in invoice.py
@moitabenfdz
account_invoice_merge : rename order to invoice
99b78cb
@moitabenfdz
account_invoice_merge: update dependance management
a6b8830
@moitabenfdz
account_invoice_merge: use commercial_partner_id instead of partner_id
b016c52
@moitabenfdz
account_invoice_merge: revert last commit (use of commercial_partner_id)
776f1a1
@moitabenfdz
account_invoice_merge: refactor do_merge method and some cleanup.
425f975
@rdeheele @moitabenfdz
[IMP] merge must linked order lines and new invoice lines (many2many …
18dd8aa 
…: table sale_order_line_invoice_rel) : the field 'invoiced' on sale.order.line needs it to provide correct information
@moitabenfdz
account_invoice_merge: update some labels
054b140
@ddico @moitabenfdz
Recreate link with invoices originating from account_analytic_line (e…
da393e3 
…g. Timesheet invoice)
@moitabenfdz
[FIX] invoice_id is not guaranteed to be in analytic line's columns
951d9a4 
	as it is added in the non-dependency hr_timesheet_invoice
@pedrobaeza @moitabenfdz
Fix PEP8
40906fd
@gurneyalex @moitabenfdz
set addons as unported to 8.0
73334cc
@moitabenfdz
Remove account_invoice_merge from __unported__
f491820
@moitabenfdz
Port to Odoo 8.0 and new api
b89c585
@moitabenfdz
Remove deprecated nsvc package
901fef3
@moitabenfdz
Replace methods returning a static tuple by variables
352e690
@moitabenfdz
Fix flake 8 issues
88bfa6e
@moitabenfdz
Fix pylint issues
f196831
@moitabenfdz
Change except_orm with Warning argument by exceptions.Warning
8befca6
@pedrobaeza @moitabenfdz
[IMP] account_invoice_merge: README.rst
8905c54
@moitabenfdz
Forward port 7.0 to 8.0
06a1cb6
@moitabenfdz
PEP8 issue
dd0427a
@moitabenfdz
Remove useless context copy
695c00b
@moitabenfdz
No need to browse after search in new API
4139f36
Sylvain Van Hoof and others added 21 commits December 21, 2023 13:47
@moitabenfdz
[MIG] account_invoice_merge: Migration to 12.0
e2ea88c
@mourad-ehm @moitabenfdz
[IMP] Make the function to get draft invoices & _dirty_check overridable
fa5c0ea
@sysadminmatmoz @moitabenfdz
Translated using Weblate (Slovenian)
b26114a 
Currently translated at 100.0% (20 of 20 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_merge/sl/
@anasuarez1 @moitabenfdz
Translated using Weblate (Spanish)
5c9fae6 
Currently translated at 90.0% (18 of 20 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_merge/es/
@legalsylvain @moitabenfdz
[FIX] account_invoice_merge: compatibility with account_invoice_margi…
ac1cf5f 
…n module (OCA/margin-analysis repository)
@aaltinisik @moitabenfdz
Translated using Weblate (Turkish)
9880182 
Currently translated at 100.0% (20 of 20 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_merge/tr/
@moitabenfdz
Translated using Weblate (Italian)
7d16ca6 
Currently translated at 100.0% (20 of 20 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_invoice_merge/it/
@fredzamoabg @moitabenfdz
[IMP] account_invoice_merge: black, isort, prettier
e58b93c
@fredzamoabg @moitabenfdz
[MIG] account_invoice_merge: Migration to 14.0
b3e2f43
@Freni-OSI @moitabenfdz
[15.0][MIG] account_invoice_merge
55f17d6
@nguyenminhchien @moitabenfdz
[16.0][MIG] account_invoice_merge: Migrate to version 16.0
25a1ba5
@mymage @moitabenfdz
Translated using Weblate (Italian)
d1b031e 
Currently translated at 80.0% (16 of 20 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-account_invoice_merge/it/
@badbole @moitabenfdz
Translated using Weblate (Croatian)
4933eb0 
Currently translated at 100.0% (20 of 20 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-account_invoice_merge/hr/
@mymage @moitabenfdz
Translated using Weblate (Italian)
f30a175 
Currently translated at 80.0% (16 of 20 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-account_invoice_merge/it/
@sysadminmatmoz @moitabenfdz
Translated using Weblate (Slovenian)
c9f62b5 
Currently translated at 80.0% (16 of 20 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-account_invoice_merge/sl/
@mymage @moitabenfdz
Translated using Weblate (Italian)
d036d88 
Currently translated at 80.0% (16 of 20 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-account_invoice_merge/it/
@Gelo-fl @moitabenfdz
Translated using Weblate (Spanish)
f741f70 
Currently translated at 100.0% (20 of 20 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-account_invoice_merge/es/
@RodrigoBM @moitabenfdz
[FIX] account_invoice_merge: this commit fixes the incorrect link bet…
2ee2840 
…ween sales orders and their invoices, and between purchase orders and their invoices.
@adrianojprado @moitabenfdz
Translated using Weblate (Portuguese (Brazil))
8b18c7e 
Currently translated at 100.0% (20 of 20 strings)

Translation: account-invoicing-16.0/account-invoicing-16.0-account_invoice_merge
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-16-0/account-invoicing-16-0-account_invoice_merge/pt_BR/
@moitabenfdz
[IMP] account_invoice_merge: pre-commit stuff
06cd681
@moitabenfdz
[MIG] account_invoice_merge: Migration to 17.0
e5d846b
@moitabenfdz moitabenfdz mentioned this pull request Dec 21, 2023
Open
46 tasks
anmarmo1
anmarmo1 approved these changes Dec 28, 2023
View reviewed changes
Copy link
Contributor

@anmarmo1 anmarmo1 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 is ok

cormaza
cormaza approved these changes Jan 13, 2024
View reviewed changes
Copy link

@cormaza cormaza left a comment

Choose a reason for hiding this comment

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

LGTM

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

@Codeleiche
Copy link

Any news on that? How could I help on the way of getting it merged?

Jortolsa-S73
Jortolsa-S73 approved these changes Dec 22, 2024
View reviewed changes
Copy link

@Jortolsa-S73 Jortolsa-S73 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

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

@cormaza cormaza cormaza approved these changes

@Jortolsa-S73 Jortolsa-S73 Jortolsa-S73 approved these changes

@anmarmo1 anmarmo1 anmarmo1 approved these changes

@MarwanBHL MarwanBHL MarwanBHL 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.