Skip to content

Fix errors summary error rate and occurrences sort alignment#241

Merged
unjica merged 2 commits into
developfrom
fix/errors-summary-status-sort
Jul 7, 2026
Merged

Fix errors summary error rate and occurrences sort alignment#241
unjica merged 2 commits into
developfrom
fix/errors-summary-status-sort

Conversation

@unjica

@unjica unjica commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Scope summary events_count to sessions with matching filtered error groups when status or message search is active (fixes error-rate KPI with Open/Resolved filters).
  • Route sort=occurrences through the aggregate SQL path so ordering matches the in-range Count column.

Test plan

  • API unit tests added/updated
  • Filter errors by Open — error rate should reflect filtered scope
  • Sort by Occurrences — row order matches Count column

Made with Cursor

unjica and others added 2 commits July 7, 2026 19:38
Scope event counts to filtered error sessions when status or search is active, and sort by in-range occurrence counts instead of lifetime totals.

Co-authored-by: Cursor <cursoragent@cursor.com>
Occurrences sort uses the aggregate SQL path, so it is no longer a ScalarErrorListSort.

Co-authored-by: Cursor <cursoragent@cursor.com>
@unjica unjica merged commit fd0e79a into develop Jul 7, 2026
3 checks passed
@unjica unjica deleted the fix/errors-summary-status-sort branch July 7, 2026 17:43
Dari0o pushed a commit to Dari0o/telemetry-tracker that referenced this pull request Jul 7, 2026
Ship errors analytics (Telemetry-Tracker#182), summary filter fixes (Telemetry-Tracker#241), and analytics window alignment.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant