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.

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

feat: Salary Revise Tool #1433

Closed

Conversation

CodeSteaks
Copy link

Salary Revise Tool: Streamline Salary Adjustments in Frappe ERP

Introduction:

This pull request introduces the Salary Revise Tool, a comprehensive solution for managing salary revisions within Frappe ERP. This tool addresses the challenge of revising employees salaries in bulk by providing Salary Revise Tool where user can fetch the existing records of the employees according to the filter given and update their salaries in bulk.

Technical Details:

  • The tool is implemented using frappe.

Integration and Usage:

  • This tool is primarily targeted towards HR Manager.

Benefits:

  • The user will be able to save a lot of time by creating salary structure assignments for employees.
  • This tool has the capability to bulk update the salaries of the employees.

Screenshots:

image

Testing Results:

Testing was completed manually. Tested each filter and also tested the speed of the tool.

Future Enhancements:

N/A

@ruchamahabal
Copy link
Member

Thanks for taking the time to contribute this feature but unfortunately we won't be able to merge it to the core since the tool's design and code is not as per product standards. We already merged the Salary Structure Assignment Tool
https://frappehr.com/docs/v14/en/salary-structure-assignment-tool

that lets you create assignments in bulk. We plan to extend the same to make edits easier too. Besides this will need more handling in terms of not allowing salary structure updates after payroll has been run for them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants