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

fix: Incorrect Earned Leaves Proration #35156

Conversation

JunKangChin
Copy link

If today date is after or equal to employee's joining date, current month should be considered in earned leaves proration.

Test Cases

  • Today Date: 1, Joining Date: 5, Result: +0
  • Today Date: 5, Joining Date: 5, Result: +1
  • Today Date: 24, Joining Date: 5, Result: +1

Consider current month for earned leaves if today date is after joining date
@stale
Copy link

stale bot commented May 19, 2023

This pull request has been automatically marked as inactive because it has not had recent activity. It will be closed within 3 days if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing.

@stale stale bot added the inactive label May 19, 2023
@ruchamahabal ruchamahabal merged commit dc04b24 into frappe:version-13-hotfix May 21, 2023
7 checks passed
@ruchamahabal
Copy link
Member

Already fixed in https://github.com/frappe/hrms

frappe-pr-bot pushed a commit that referenced this pull request May 24, 2023
## [13.50.6](v13.50.5...v13.50.6) (2023-05-24)

### Bug Fixes

* allow over-payment against SO ([#35079](#35079)) ([eb243c2](eb243c2))
* bypass flag in Customer Group wasn't effective ([f0c9d89](f0c9d89))
* change field-type to remove currency field from total row in export ([f65be40](f65be40))
* consider 0 if rate/qty are null (backport [#35338](#35338)) ([#35341](#35341)) ([387f8b9](387f8b9))
* depreciation schedule for existing assets [v14] (backport [#35255](#35255)) ([#35347](#35347)) ([7506132](7506132))
* error while saving job card ([d6427cf](d6427cf))
* get_query filters ([2aa7729](2aa7729))
* Incorrect Earned Leaves Proration ([#35156](#35156)) ([dc04b24](dc04b24))
* linter ([0a42e6f](0a42e6f))
* non manufacturing items/fixed asset items in BOM ([66ba74f](66ba74f))
* Pick List TypeError ([137898d](137898d))
* tds incorrectly calculated for invoice that are below threshold ([6c170ab](6c170ab))
* **test:** cumulative threshold checks ([06deecb](06deecb))
* use flt instead of mandatory field ([f63b866](f63b866))
@frappe-pr-bot
Copy link
Collaborator

🎉 This PR is included in version 13.50.6 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants