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

[16.0][IMP] payroll: Show payroll inputs menu to allow faster input edit #182

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from
Open

[16.0][IMP] payroll: Show payroll inputs menu to allow faster input edit #182

wants to merge 1 commit into from

Conversation

giarve
Copy link

@giarve giarve commented Jan 12, 2025

In config menu.

@OCA-git-bot
Copy link
Contributor

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

@giarve giarve force-pushed the 16.0-hr_payroll_inputs branch 2 times, most recently from 16531b1 to dd776fd Compare January 12, 2025 16:01
@giarve giarve force-pushed the 16.0-hr_payroll_inputs branch from dd776fd to 6b90dbd Compare January 12, 2025 16:04
@nimarosa
Copy link
Contributor

/ocabot merge patch

@giarve
Copy link
Author

giarve commented Feb 18, 2025 •
edited
Loading

Taking advantage of the fact that the bot did not merge yet, I found that the payroll input is not multi-company compatible (it will show all payslip input records from another company).

If anybody wants to suggest changes to fix this, please go ahead or merge it but take into account that it may leak data from another company

EDIT: actually hr.payslip.run has the same problen in 16.0

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

@dreispt dreispt dreispt approved these changes

@marcelsavegnago marcelsavegnago marcelsavegnago approved these changes

@nimarosa nimarosa nimarosa approved these changes

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

Successfully merging this pull request may close these issues.
5 participants
@giarve @OCA-git-bot @nimarosa @dreispt @marcelsavegnago