## Description Adds drilldown support for report bar charts powered by `ReportContainer`. Clicking a non-zero report bar now opens a right-side drawer with the conversations or messages that contributed to that bucket, with each row linking to the underlying conversation and message rows linking with `messageId`. This includes a new `GET /api/v2/accounts/:account_id/reports/drilldown` endpoint, backend drilldown builders/serializers, generic chart click emission, local drawer state via `useReportDrilldown`, compact drilldown cards, pagination, stale-response protection, and validation for unsupported drilldown dimensions. Fixes # CW-4497 https://linear.app/chatwoot/issue/CW-4497/drill-down-on-agent-conversations-report ## Type of change - [x] Bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality not to work as expected) - [ ] This change requires a documentation update ## How Has This Been Tested? Ran the focused backend and frontend checks for the drilldown endpoint, builder, chart click handling, drawer/card UI, API helper, and stale-response handling. Here are the screenshots on how it looks like: <img width="1792" height="1199" alt="Screenshot 2026-06-02 at 11 32 11 PM" src="https://github.com/user-attachments/assets/6bdb8832-b9df-4bf3-9a2a-beaefe203b6e" /> <img width="1791" height="1230" alt="Screenshot 2026-06-02 at 11 32 34 PM" src="https://github.com/user-attachments/assets/36e92eb7-3208-4855-87f4-0c7f316df54d" /> <img width="1784" height="1235" alt="Screenshot 2026-06-02 at 11 32 46 PM" src="https://github.com/user-attachments/assets/f7a53916-74f2-4622-9305-042e0ac9e877" /> ## Checklist: - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my code - [x] I have commented on my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [x] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged and published in downstream modules --------- Co-authored-by: Vishnu Narayanan <iamwishnu@gmail.com> Co-authored-by: Shivam Mishra <scm.mymail@gmail.com>
54 lines
1.6 KiB
Ruby
54 lines
1.6 KiB
Ruby
module Reports::DrilldownTimestampValidator
|
|
extend TimezoneHelper
|
|
|
|
TIMESTAMP_PARAMS = %i[bucket_timestamp since until].freeze
|
|
DEFAULT_GROUP_BY = V2::Reports::DrilldownBuilder::DEFAULT_GROUP_BY
|
|
SUPPORTED_GROUP_BY = V2::Reports::DrilldownBuilder::SUPPORTED_GROUP_BY
|
|
|
|
module_function
|
|
|
|
def valid?(params)
|
|
timestamps = TIMESTAMP_PARAMS.index_with { |param| integer_param(params[param]) }
|
|
return false if timestamps.values.any?(&:nil?)
|
|
return false unless timestamps[:since] < timestamps[:until]
|
|
|
|
bucket_overlaps_requested_range?(params, timestamps)
|
|
end
|
|
|
|
def integer_param(value)
|
|
return unless value.to_s.match?(/\A\d+\z/)
|
|
|
|
value.to_i
|
|
end
|
|
|
|
def bucket_overlaps_requested_range?(params, timestamps)
|
|
bucket_start = Time.zone.at(timestamps[:bucket_timestamp]).in_time_zone(timezone(params))
|
|
bucket_end = bucket_end_for(bucket_start, group_by(params))
|
|
requested_start = Time.zone.at(timestamps[:since])
|
|
requested_end = Time.zone.at(timestamps[:until])
|
|
|
|
bucket_start < requested_end && bucket_end > requested_start
|
|
rescue ArgumentError, RangeError
|
|
false
|
|
end
|
|
|
|
def bucket_end_for(bucket_start, group_by)
|
|
{
|
|
'hour' => bucket_start + 1.hour,
|
|
'day' => bucket_start + 1.day,
|
|
'week' => bucket_start + 1.week,
|
|
'month' => bucket_start + 1.month,
|
|
'year' => bucket_start + 1.year
|
|
}.fetch(group_by)
|
|
end
|
|
|
|
def group_by(params)
|
|
group = params[:group_by].to_s
|
|
SUPPORTED_GROUP_BY.include?(group) ? group : DEFAULT_GROUP_BY
|
|
end
|
|
|
|
def timezone(params)
|
|
timezone_name_from_offset(params[:timezone_offset])
|
|
end
|
|
end
|