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

[18.0][MIG] payroll_account: Migration to 18.0 #185

Open
wants to merge 51 commits into
base: 18.0
Choose a base branch
from
Open

[18.0][MIG] payroll_account: Migration to 18.0 #185

wants to merge 51 commits into from

Conversation

dreispt
Copy link
Member

@dreispt dreispt commented Jan 27, 2025 •
edited
Loading

No description provided.

@dreispt dreispt force-pushed the 18.0-mig-payroll_account branch 9 times, most recently from 618b745 to babf857 Compare February 1, 2025 21:06
@nimarosa
Copy link
Contributor

@dreispt Tests are failing here. Is this PR ready for review ?

Saran440 and others added 20 commits February 23, 2025 18:26
@Saran440 @dreispt
rename hr_payroll_account to payroll_account
f8d08e9
@OCA-git-bot @dreispt
[ADD] icon.png
1916805
@MiquelRForgeFlow @dreispt
[ADD] payroll,payroll_account: migration analysis
0fcb4f3
@norlinhenrik @dreispt
[14.0][MIG] payroll_account: Migration to 14.0
d31358a
@dreispt
[FIX] payroll_account: Add taxes to account.move.line
8c7e6b6
@Noel000 @dreispt
Translated using Weblate (Catalan)
7c3a1ce 
Currently translated at 100.0% (24 of 24 strings)

Translation: payroll-14.0/payroll-14.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-14-0/payroll-14-0-payroll_account/ca/
@ibuioli @dreispt
Added translation using Weblate (Spanish (Argentina))
dbc9ae0
@ibuioli @dreispt
Translated using Weblate (Spanish (Argentina))
bf2d603 
Currently translated at 16.6% (4 of 24 strings)

Translation: payroll-14.0/payroll-14.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-14-0/payroll-14-0-payroll_account/es_AR/
@ibuioli @dreispt
Translated using Weblate (Spanish (Argentina))
9cc8d87 
Currently translated at 100.0% (24 of 24 strings)

Translation: payroll-14.0/payroll-14.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-14-0/payroll-14-0-payroll_account/es_AR/
@OCA-git-bot @dreispt
[IMP] update dotfiles [ci skip]
f14f953
@nimarosa @dreispt
[IMP] payroll_account: prefer groups in rules views
aa112ac
@nimarosa @dreispt
[IMP] payroll_account: add nimarosa to manteiners
9730f0b
@MiquelRForgeFlow @dreispt
[FIX] payroll_account: post() -> action_post()
c002ec7
@nimarosa @dreispt
Translated using Weblate (Spanish)
2b6a632 
Currently translated at 100.0% (24 of 24 strings)

Translation: payroll-14.0/payroll-14.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-14-0/payroll-14-0-payroll_account/es/
@mtelahun @dreispt
[FIX] payroll_account: log a warning if no move lines are generated f…
ad57c9c 
…or the payslip

Until now this module tried to confirm an empty account move. This caused
the account module to throw an exception. This causes problems in
other modules that may not know about payroll_account. For example
in other modules' tests.
@nimarosa @dreispt
[IMP] payroll: Merge payslip_cancel in base module and remove module
f46d61f
@ibuioli @dreispt
Translated using Weblate (Spanish (Argentina))
676e485 
Currently translated at 100.0% (24 of 24 strings)

Translation: payroll-14.0/payroll-14.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-14-0/payroll-14-0-payroll_account/es_AR/
@nimarosa @dreispt
Translated using Weblate (Spanish)
cc3abbe 
Currently translated at 100.0% (24 of 24 strings)

Translation: payroll-14.0/payroll-14.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-14-0/payroll-14-0-payroll_account/es/
@nimarosa @dreispt
[14.0] [IMP] payroll: improve salary_rule and categories views and do…
3ec61dd 
…cumentation

[14.0] [IMP] payroll: improve salary rules views and documentation

[14.0] [IMP] payroll: improve salary rules  and categories views

[14.0] [IMP] payroll: improve salary rules views and documentation

[14.0] [IMP] payroll: improve salary rules views and documentation

[IMP] payroll: fix repeated words

[14.0] [IMP] payroll: fix typos
@nimarosa @dreispt
[14.0] [IMP] payroll: change manifest category
9533c22 
[14.0] [IMP] payroll: change manifest category

[14.0] [IMP] payroll: add migration for new payslip and payslips objects

[14.0] [IMP] payroll: add migration for new payslip and payslips objects

[14.0] [IMP] payroll: add migration for new payslip and payslips objects
OCA-git-bot and others added 27 commits February 23, 2025 18:26
@OCA-git-bot @dreispt
[UPD] README.rst
4c49482
@dreispt
[UPD] Update payroll_account.pot
7bdb3d0
@weblate @dreispt
Update translation files
cddfa80 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/
@mostafabarmshory @dreispt
Translated using Weblate (Persian)
a6ae84b 
Currently translated at 100.0% (21 of 21 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/fa/
@mymage @dreispt
Translated using Weblate (Italian)
bedcffe 
Currently translated at 80.9% (17 of 21 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/it/
@OCA-git-bot @dreispt
[UPD] README.rst
58ca20b
@hussain @dreispt
Translated using Weblate (Arabic)
af893b2 
Currently translated at 85.7% (18 of 21 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/ar/
@douglascstd @dreispt
Translated using Weblate (Portuguese (Brazil))
6d886d6 
Currently translated at 85.7% (18 of 21 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/pt_BR/
@mymage @dreispt
Translated using Weblate (Italian)
4106df1 
Currently translated at 80.9% (17 of 21 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/it/
@weblate @dreispt
Update translation files
6e21ef4 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/
@mymage @dreispt
Translated using Weblate (Italian)
7bd9cf5 
Currently translated at 100.0% (21 of 21 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/it/
@cvinh @dreispt
[FIX] Fix analytic account in payroll account
3eb187d
@OCA-git-bot @dreispt
[BOT] post-merge updates
5a2289e
@dreispt
Translated using Weblate (Portuguese (Brazil))
3ddb1c0 
Currently translated at 100.0% (21 of 21 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/pt_BR/
@FernandoRomera @dreispt
[14.0][FIX] multicompanies
ea7facf
@FernandoRomera @dreispt
[14.0][FIX] payroll_account: Fix multicompanies and refactoring
35ed08c
@marcelsavegnago @dreispt
[FIX] payroll_account: fix company_dependent on debit and credit account
c13440d
@norlinhenrik @dreispt
[FIX] payroll_account: hr.payslip.line _get_partner_id()
8420272
@dreispt
[UPD] Update payroll_account.pot
10716bb
@weblate @dreispt
Update translation files
202c822 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/
@dreispt
Translated using Weblate (Portuguese (Brazil))
d5a2e9b 
Currently translated at 100.0% (22 of 22 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/pt_BR/
@mymage @dreispt
Translated using Weblate (Italian)
184020a 
Currently translated at 100.0% (22 of 22 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/it/
@peterromao @dreispt
Translated using Weblate (Portuguese)
5832a81 
Currently translated at 100.0% (22 of 22 strings)

Translation: payroll-16.0/payroll-16.0-payroll_account
Translate-URL: https://translation.odoo-community.org/projects/payroll-16-0/payroll-16-0-payroll_account/pt/
@dreispt
[IMP] payroll_account: pre-commit execution
f52fe30
@dreispt
[MIG] payroll_account: Migration to 18.0
edf4bad
@dreispt
[FIX] payroll: error testing report printing
ffc2cd3
@dreispt
[FIX] payroll_account: improve test coverage, remove redundant code
186a03a
@dreispt dreispt force-pushed the 18.0-mig-payroll_account branch from babf857 to 186a03a Compare February 23, 2025 18:26
@dreispt
Copy link
Member Author

dreispt commented Feb 23, 2025

@nimarosa I rebased and tests are passing now.

@dreispt dreispt requested a review from nimarosa February 23, 2025 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nimarosa nimarosa Awaiting requested review from nimarosa

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.