Skip to content

[REF] hr_payroll_document: In tests, extract common setup - #288

Open
SirPyTech wants to merge 4 commits into
OCA:18.0from
PyTech-SRL:18.0-ref-hr_payroll_document-tests_common
Open

[REF] hr_payroll_document: In tests, extract common setup#288
SirPyTech wants to merge 4 commits into
OCA:18.0from
PyTech-SRL:18.0-ref-hr_payroll_document-tests_common

Conversation

@SirPyTech

@SirPyTech SirPyTech commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Porting to 18.0 the improvements to hr_payroll_document already merged in 16.0 with #236.

They are already being proposed in 17.0 with #241.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @peluko00,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added series:18.0 mod:hr_payroll_document Module hr_payroll_document labels Aug 25, 2026

@CristianoMafraJunior CristianoMafraJunior left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

niceeee

@SirPyTech
SirPyTech force-pushed the 18.0-ref-hr_payroll_document-tests_common branch from f43d62c to 89625ff Compare August 27, 2026 09:22
@SirPyTech

Copy link
Copy Markdown
Contributor Author

Thanks @CristianoMafraJunior for your review, I added a few more commits because I need the whole set of changes that was merge in 16.0 with #236, please update your review.

@SirPyTech
SirPyTech marked this pull request as draft August 27, 2026 09:44
@SirPyTech
SirPyTech force-pushed the 18.0-ref-hr_payroll_document-tests_common branch from 89625ff to 173cc42 Compare August 27, 2026 09:48
@SirPyTech
SirPyTech marked this pull request as ready for review August 27, 2026 09:49

@HekkiMelody HekkiMelody left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code review, LGTM

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants