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: fetch leave encashment amount from Salary Structure Assignment #2256

Closed
wants to merge 77 commits into from

Conversation

iamejaaz
Copy link
Contributor

@iamejaaz iamejaaz commented Oct 6, 2024

Closes #1275

Please provide enough information so that others can review your pull request:

Reference #1275
#1287 (comment)

As suggested in this PR (#1287), fetch the amount from the salary structure assignment instead of the salary structure.
image

Explain the details for making this change. What existing problem does the pull request solve?

It will allow us to set employee-specific encashment amounts, just like different base and variable amounts.

Screenshots/GIFs

The amount will be fetched from the Salary Structure in the Salary Structure Assignment Doctype if it is empty; otherwise, the user can manually enter the amount.
image

Leave Encashment
image

@krantheman
Copy link
Member

Please fix formatting and linting.

krantheman and others added 27 commits October 16, 2024 17:42
@iamejaaz
Copy link
Contributor Author

iamejaaz commented Oct 16, 2024

Please fix formatting and linting.

While editing, I encountered conflicts with the commits, so I created a new PR and closing this PR
#2296

@iamejaaz iamejaaz closed this Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: allow Leave Encashment amount to be editable
5 participants