-
-
Notifications
You must be signed in to change notification settings - Fork 351
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.
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
[RFC] hr_employee_cost_history: add a comment field #696
Comments
Should that be an HR contract or some paperwork attached? |
Not necessarily. For instance in Belgium & Luxemburg there is an automatic salary indexation mechanism without any contractual change. |
It makes sense indeed. Ping me in your PR to review it! Thank you! |
I'll test it too @sbidoul 👍🏼 |
There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
Done in #735. |
@sbidoul you can close this one. PR is merging. Thank you! |
Thanks for the hr_employee_cost_history module.
Would it make sense to add a comment field on history line?
The use case is to document the reason for the change (salary raise, automatic indexation, etc).
If you agree, we can do a PR.
@rafaelbn @edlopen
The text was updated successfully, but these errors were encountered: