Skip to content

Conversation

@teeohhem
Copy link
Contributor

@teeohhem teeohhem commented Nov 7, 2025

Fixes: HDX-2789

@changeset-bot
Copy link

changeset-bot bot commented Nov 7, 2025

🦋 Changeset detected

Latest commit: ad22c96

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@hyperdx/app Patch
@hyperdx/api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Nov 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
hyperdx-v2-oss-app Ready Ready Preview Comment Nov 7, 2025 7:42pm

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

E2E Test Results

1 test failed • 39 passed • 3 skipped • 376s

Status Count
✅ Passed 39
❌ Failed 1
⚠️ Flaky 0
⏭️ Skipped 3

View full report →

@claude
Copy link

claude bot commented Nov 7, 2025

Code Review

No critical issues found.

The implementation looks good:

  • ✅ Properly imports useEffect from React (already in imports)
  • ✅ Correct dependency array [where] for the effect
  • ✅ Simple, clear logic that switches to "all" when filters are applied
  • ✅ E2E test coverage validates the behavior
  • ✅ Follows project patterns (functional components, React hooks)

Minor note: Consider whether this behavior should persist when filters are cleared (currently it does not reset back to "running"), but this may be intentional UX design.

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.

2 participants