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: process leave encashment via payment entry #2775

Open
wants to merge 15 commits into
base: develop
Choose a base branch
from

Conversation

AyshaHakeem
Copy link
Collaborator

@AyshaHakeem AyshaHakeem commented Feb 11, 2025

Previously, Leave Encashment was always processed by creating an Additional Salary on submission, which was later paid through the Salary Slip.

  • Add an option to process Leave Encashment via a Payment Entry instead of a Salary Slip.
  • Introduced a status field to track payment status.
  • Status now updates when paid via Payment Entry or Salary Slip.

Patch to update payment status of existing, submitted leave encashment docs based on whether they have been paid via salary slip

Before:
No status tracking, only Salary Slip-based payment.
Screenshot 2025-02-11 at 12 26 51 PM

After:
leave-payment

@AyshaHakeem AyshaHakeem marked this pull request as draft February 11, 2025 07:09
@AyshaHakeem AyshaHakeem marked this pull request as draft February 11, 2025 07:09
@AyshaHakeem AyshaHakeem marked this pull request as draft February 11, 2025 07:09
@AyshaHakeem AyshaHakeem marked this pull request as draft February 11, 2025 07:09
@AyshaHakeem
Copy link
Collaborator Author

#2609

Copy link
Contributor

@mohsinalimat mohsinalimat left a comment

Choose a reason for hiding this comment

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

hrms/patches/v15_0/update_payment_status_for_leave_encashment

It's a .py file. Please add extension.

@AyshaHakeem AyshaHakeem removed the request for review from ruchamahabal February 11, 2025 19:29
@AyshaHakeem AyshaHakeem marked this pull request as ready for review February 20, 2025 06:07
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.

2 participants