-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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(Issue): Calculate first_response_time based on working hours #25991
Merged
nextchamp-saqib
merged 19 commits into
frappe:develop
from
GangaManoj:fix-frt-calculation
Jul 14, 2021
Merged
fix(Issue): Calculate first_response_time based on working hours #25991
nextchamp-saqib
merged 19 commits into
frappe:develop
from
GangaManoj:fix-frt-calculation
Jul 14, 2021
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
GangaManoj
requested review from
nextchamp-saqib
and removed request for
marination
June 9, 2021 00:29
GangaManoj
force-pushed
the
fix-frt-calculation
branch
from
June 9, 2021 00:31
b46191d
to
533416b
Compare
GangaManoj
force-pushed
the
fix-frt-calculation
branch
from
June 21, 2021 16:38
c3ea3b1
to
533416b
Compare
GangaManoj
force-pushed
the
fix-frt-calculation
branch
from
June 21, 2021 16:55
533416b
to
0a9271d
Compare
nextchamp-saqib
added
CI-failing
Unit tests or patch tests are failing.
fix-sider-issues
Check and fix issues listed by Sider
labels
Jun 23, 2021
nextchamp-saqib
removed
fix-sider-issues
Check and fix issues listed by Sider
CI-failing
Unit tests or patch tests are failing.
labels
Jun 24, 2021
nextchamp-saqib
requested changes
Jun 24, 2021
GangaManoj
force-pushed
the
fix-frt-calculation
branch
from
June 24, 2021 12:00
e4009ce
to
e5413bc
Compare
GangaManoj
force-pushed
the
fix-frt-calculation
branch
from
June 29, 2021 10:59
aa26435
to
6978e07
Compare
nextchamp-saqib
requested changes
Jul 9, 2021
Co-authored-by: Saqib <nextchamp.saqib@gmail.com>
nextchamp-saqib
requested changes
Jul 13, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem:
First Response Time calculation does not take working hours into consideration- it just takes the difference between first_responded_on and time of creation of the Issue.
Fix:
Checks if first_responded_on and time of creation of the Issue are: