Skip to content

[16.0][IMP] payroll: action_clear_employees#257

Open
CristianoMafraJunior wants to merge 1 commit into
OCA:16.0from
Escodoo:16.0-imp-action_clear_employees
Open

[16.0][IMP] payroll: action_clear_employees#257
CristianoMafraJunior wants to merge 1 commit into
OCA:16.0from
Escodoo:16.0-imp-action_clear_employees

Conversation

@CristianoMafraJunior

Copy link
Copy Markdown
Member

@Escodoo HT01879

Why

In the Generate Payslips wizard (hr.payslip.employees), the employee list can open with hundreds of pre-loaded records, which slows down the UI and makes it hard to pick only the employees that should be included in the batch. Generation behavior is unchanged; what was missing was a quick way to clear the full selection without deleting rows one by one.

What changed

Added action_clear_employees on transient model hr.payslip.employees, clearing employee_ids using Many2many command (5, 0, 0) and keeping the wizard open.
Added a header button on the wizard form that calls action_clear_employees, so users can remove all selected employees in one click.
The Generate button still requires at least one employee (existing check in compute_sheet).

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@CristianoMafraJunior CristianoMafraJunior force-pushed the 16.0-imp-action_clear_employees branch from 48ae6a7 to 26a106f Compare March 30, 2026 17:31
@CristianoMafraJunior CristianoMafraJunior marked this pull request as draft April 8, 2026 18:35
@CristianoMafraJunior CristianoMafraJunior force-pushed the 16.0-imp-action_clear_employees branch from 26a106f to e5d6ff9 Compare April 13, 2026 13:20
@CristianoMafraJunior CristianoMafraJunior force-pushed the 16.0-imp-action_clear_employees branch from e5d6ff9 to 2ae9539 Compare April 14, 2026 11:45
@CristianoMafraJunior CristianoMafraJunior marked this pull request as ready for review April 28, 2026 12:43
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.

2 participants