Skip to content

Conversation

@bokelley
Copy link
Contributor

Summary

  • Fixed bug where "Needs Attention" tab showed 11 items but table showed "No accounts found"
  • The view count queries were using different filtering logic than the results queries
  • Removed a duplicate endpoint that was unreachable dead code

Changes

  • needs_attention: Count now correctly separates non-members with action items from members with expiring subscriptions (matches results query)
  • new_insights: Count now uses 30-day window (was 7 days, results used 30)
  • hot: Count now uses OR logic for engagement/interest (was AND, results used OR)
  • Removed 158 lines of dead duplicate /accounts/view-counts endpoint

Test plan

  • Verified with Vibium that counts now match results
  • All tests pass
  • TypeScript compiles

🤖 Generated with Claude Code

The view counts endpoint was showing different numbers than actual results
because the count queries used different filtering logic than the results
queries:

- needs_attention: Count included all orgs with action items, but results
  only showed non-members OR members with expiring subscriptions
- new_insights: Count used 7-day window, results used 30-day window
- hot: Count required both engagement AND interest, results used OR

Also removed a dead duplicate endpoint that was unreachable.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@bokelley bokelley merged commit af274d2 into main Jan 20, 2026
6 checks passed
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