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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: include resolution time and waiting time in team & agent report #9296
Conversation
2624cf4
to
e5e20f1
Compare
e5e20f1
to
3045773
Compare
@sojan-official need your review on this. A thing to note is that this might reverse the perf benefits we got in #8733 |
app/builders/v2/report_builder.rb
Outdated
avg_resolution_time: avg_resolution_time_summary, | ||
resolutions_count: resolutions.count, | ||
reply_time: reply_time_summary |
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.
@scmmishra Can we run the query only on the download? These would add additional query and would fail already overloaded API.
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.
The report for download is made using the generate_report
function in report_helpers which uses the short_summary
method internally. All these params are required to generate the new downloads.
One thing I can do is use the summary
method for agent and teams report and rest of the reports can use the short summary. You can see the changes in this commit 134e886
Is this what you were looking for?
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.
Please find the comment above.
No description provided.