-
Notifications
You must be signed in to change notification settings - Fork 688
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
refactor(HR Dashboard): show current quarter for Employees Joining and Employees Relieving cards #1268
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #1268 +/- ##
===========================================
- Coverage 72.43% 72.41% -0.03%
===========================================
Files 201 201
Lines 10946 10946
===========================================
- Hits 7929 7926 -3
- Misses 3017 3020 +3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this and the employees joining card should probably be for This Quarter. Relieving and Joining are mostly not decided a quarter in advance.
Will have to change the name and filters
…d Employees Relieving cards
be7cc7f
to
711211c
Compare
…d Employees Relieving cards (#1268) * refactor(HR Dashboard): show current quarter for Employees Joining and Employees Relieving cards * chore: update doc owner --------- Co-authored-by: Rucha Mahabal <[email protected]> (cherry picked from commit 589e62e)
…d Employees Relieving cards (#1268) * refactor(HR Dashboard): show current quarter for Employees Joining and Employees Relieving cards * chore: update doc owner --------- Co-authored-by: Rucha Mahabal <[email protected]> (cherry picked from commit 589e62e)
…d Employees Relieving cards (backport #1268) (#1272) * refactor(HR Dashboard): show current quarter for Employees Joining and Employees Relieving cards * chore: update doc owner --------- Co-authored-by: Rucha Mahabal <[email protected]> (cherry picked from commit 589e62e) Co-authored-by: Akash Tom <[email protected]>
…d Employees Relieving cards (backport #1268) (#1273) * refactor(HR Dashboard): show current quarter for Employees Joining and Employees Relieving cards * chore: update doc owner --------- Co-authored-by: Rucha Mahabal <[email protected]> (cherry picked from commit 589e62e) Co-authored-by: Akash Tom <[email protected]>
…d Employees Relieving cards (frappe#1268) * refactor(HR Dashboard): show current quarter for Employees Joining and Employees Relieving cards * chore: update doc owner --------- Co-authored-by: Rucha Mahabal <[email protected]>
Closes: #1181
This PR causes the Employees Joining and Employees Relieving cards to be filtered by the current quarter, instead of the next one.
no-docs