Skip to content

[18.0][FW] payroll: ported 111, 117, 119, 151 #187

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
Open
norlinhenrik wants to merge 13 commits into
base: 18.0
Choose a base branch
from
Open

[18.0][FW] payroll: ported 111, 117, 119, 151 #187

norlinhenrik wants to merge 13 commits into from

Conversation

@norlinhenrik
Copy link
Contributor

@norlinhenrik norlinhenrik commented Jan 27, 2025

No description provided.

nimarosa and others added 13 commits January 27, 2025 13:49
@nimarosa @norlinhenrik
[14.0] [IMP] payroll: Add sum_category payslips browsable method
55d156a
@nimarosa @norlinhenrik
[14.0] [IMP] payroll: Add datetime to tools object
fdf7b11
@nimarosa @norlinhenrik
[14.0] [IMP] payroll: Add "category_max" and "rule_max" method
c61430b
@nimarosa @norlinhenrik
[14.0] [IMP] payroll: Add "category_max" and "rule_max" method
6facc71
@nimarosa @norlinhenrik
[14.0] [IMP] payroll: Add "min_category" and "min_rule" method
ab64c88
@nimarosa @norlinhenrik
[14.0] [IMP] payroll: Add "average_category" and "average_rule"
5194549
@nimarosa @norlinhenrik
[14.0] [IMP] payroll: Rename sum() to sum_rule() and add deprecation …
f50f2e0 
…warning
@nimarosa @norlinhenrik
[14.0] [IMP] payroll: Add min/max/average monthly methods
5aa2e06
@nimarosa @norlinhenrik
[14.0] [IMP] payroll: Add group to controls buttons
c2385bf
@norlinhenrik
[IMP] payroll: salary rule code: payslip.line_sum_where()
fa5d4e2
@norlinhenrik
[IMP] payroll: localdict rule.code or rule.id; test line_sum_where()
f630b59
@ivancandelas @norlinhenrik
[FIX] payroll: show code field on hr.leave.type view
0bb196a
@kasado @norlinhenrik
[FIX] Payslip item name in user interface language.
9dce4c0
@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 27, 2025

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

@nimarosa
Copy link
Contributor

nimarosa commented Jan 27, 2025

@norlinhenrik please check pre commit so we can merge

@nimarosa
Copy link
Contributor

nimarosa commented May 6, 2025

@norlinhenrik Are we moving foward with this PR? i think you need to rebase and also check pre-commit errors.

@github-actions
Copy link

github-actions bot commented Sep 7, 2025

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Sep 7, 2025
gbrito
gbrito reviewed Sep 19, 2025
View reviewed changes
Copy link

@gbrito gbrito left a comment

Choose a reason for hiding this comment

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

would be nice to run the pre-commit, I could do it and push it

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Sep 21, 2025
@github-actions
Copy link

github-actions bot commented Jan 25, 2026

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@gbrito gbrito gbrito approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@norlinhenrik @OCA-git-bot @nimarosa @gbrito @ivancandelas @kasado