Skip to content

[ADD] hr_payroll_document_queue - #292

Open
SirPyTech wants to merge 4 commits into
OCA:18.0from
PyTech-SRL:18.0-add-hr_payroll_document_queue
Open

[ADD] hr_payroll_document_queue#292
SirPyTech wants to merge 4 commits into
OCA:18.0from
PyTech-SRL:18.0-add-hr_payroll_document_queue

Conversation

@SirPyTech

Copy link
Copy Markdown
Contributor

Forward porting of the module already proposed in #252:

Add a new module to process the payslips asynchronously.

I have also included a small improvement to allow multiple wizards to be processed at the same time without conflicts.
A more elaborate solution could be implemented (like using https://docs.python.org/3/library/tempfile.html) but that would require more refactoring.

Let me know what you think!

@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 mod:hr_payroll_document_queue Module hr_payroll_document_queue labels Aug 27, 2026

@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

/>
<field name="arch" type="xml">
<field name="payrolls" position="after">
<field name="is_payroll_being_processed" invisible="True" />

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

chore (non-blocking): my understanding is that this (and also is_send_visible below) don't need to be added manually anymore in v18, so they could be removed to simplify the view.

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

Labels

mod:hr_payroll_document_queue Module hr_payroll_document_queue mod:hr_payroll_document Module hr_payroll_document series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants