-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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: Leave Application should not be in hook.py #36008
Conversation
I am not sure this is correct, but AFAIK, Leave Application is in hrms app.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #36008 +/- ##
========================================
Coverage 64.22% 64.22%
========================================
Files 785 785
Lines 60640 60640
========================================
Hits 38949 38949
Misses 21691 21691
|
@kittiu please also create a PR to https://github.com/frappe/hrms that adds this line there, so that we don't lose any functionality. |
Thanks and done. |
@kittiu Can you create the pull request on hrms? |
frappe/hrms#691 |
(cherry picked from commit 2607847) # Conflicts: # erpnext/hooks.py
#36158) Co-authored-by: Kitti U. @ Ecosoft <kittiu@gmail.com> Co-authored-by: Rucha Mahabal <ruchamahabal2@gmail.com>
I am not sure this is correct, but AFAIK, Leave Application should be in hrms app, is it not?