Conversion breakdowns drop every later day once the series limit is reached

Đang mở
  • Có gì không ổn?
#655 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@officialasishkumar đang làm issue này rồi.

Từ ngày 7/11/2026.

  • #658 của @officialasishkumar — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
35/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
typescript
Lĩnh vực
data-visualization

Hướng nghiên cứu

Start with packages/core/src/modules/chart/conversion.service.ts and inspect toSeries where the limit is checked before the breakdown key. Compare its behavior with funnel.service.ts, whose equivalent fix is described in the issue. Done means the limit caps the number of breakdown series while every kept series retains all days in the range; pull request #658 is already open.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Description

A conversion chart with a breakdown loses data once the number of breakdown values reaches the report's limit. Every row after that point is dropped, including the later days of the series that were kept. Those lines stop early, and if the limit is reached on the first day, each series shows only that day. The summary and table built from the series are wrong in the same way.

The report editor sends limit: 500, so this happens with any breakdown that has at least 500 values over the range (paths, referrers, many custom properties). A smaller limit makes it happen sooner.

Root cause

toSeries in packages/core/src/modules/chart/conversion.service.ts checks the limit before it looks at the row's breakdown key:

if (limit && Object.keys(acc).length >= limit) {
  return acc;
}

The query orders rows by event_day, then by breakdown, so once the limit is reached, every later row is discarded, including rows of series that were already accepted.

The funnel's toSeries (funnel.service.ts) had the same bug and was fixed by checking the limit only for new keys. The conversion version never got that fix.

Expected behavior

The limit caps how many breakdown series come back. The series that are kept still have every day in the range.

Ngôn ngữ chính
TypeScript
Star
7.3k
Fork
539
Merge trung bình
1 ngày 20 giờ
Pull request đã merge (30 ngày)
80

Chuẩn bị môi trường

  • Có Dockerfile hoặc tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Openpanel-dev/openpanel

Tất cả issue của Openpanel-dev/openpanel

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.