Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(backend): support no app version filters #1732

Closed
wants to merge 2 commits into from
Closed

Conversation

detj
Copy link
Contributor

@detj detj commented Jan 16, 2025

Summary

This PR updates all relevant dashboard APIs to return empty response when requested with no app versions. This allows us to implement a "Clear all" filter option on the dashboard.

Tasks

  • Update dashboard overview metrics page
  • Modify GET /apps/:id/metrics API to support no app versions
  • Modify GET /apps/:id/journey API to support no app versions
  • Modify GET /apps/:id/crashGroups API to support no app versions
  • Modify GET /apps/:id/crashGroups/plots/instances API to support no app versions
  • Modify GET /apps/:id/crashGroups/:crashGroupId/crashes API to support no app versions
  • Modify GET /apps/:id/crashGroups/:crashGroupId/plots/instances API to support no app versions
  • Modify GET /apps/:id/crashGroups/:crashGroupId/plots/distribution API to support no app versions
  • Modify GET /apps/:id/anrGroups API to support no app versions
  • Modify GET /apps/:id/anrGroups/plots/instances API to support no app versions
  • Modify GET /apps/:id/anrGroups/:anrGroupId/anrs API to support no app versions
  • Modify GET /apps/:id/anrGroups/:anrGroupId/plots/instances API to support no app versions
  • Modify GET /apps/:id/anrGroups/:anrGroupId/plots/distribution API to support no app versions
  • Update GET /apps/:id/sessions API
  • Update GET /apps/:id/sessions/plots/instances API
  • Update GET /apps/:id/sessions/:sessionId API
  • Update GET /apps/:id/spans/roots/names API
  • Update GET /apps/:id/spans/instances API
  • Update GET /apps/:id/spans/plot API
  • Test all endpoints thoroughly
  • 📚 Update dashboard API documentation

See also

- Modify GET `/apps/:id/metrics` API to support no app versions
- Modify GET `/apps/:id/journey` API to support no app versions
- Modify GET `/apps/:id/crashGroups` API to support no app versions
- Modify GET `/apps/:id/crashGroups/plots/instances` API to support no
app versions
- Modify GET `/apps/:id/crashGroups/:crashGroupId/crashes` API to
support no app versions
- Modify GET `/apps/:id/crashGroups/:crashGroupId/plots/instances` API
to support no app versions
- Modify GET `/apps/:id/crashGroups/:crashGroupId/plots/distribution`
API to support no app versions
- Modify GET `/apps/:id/anrGroups` API to support no app versions
- Modify GET `/apps/:id/anrGroups/plots/instances` API to support no app
versions
- Modify GET `/apps/:id/anrGroups/:anrGroupId/anrs` API to support no
app versions
- Modify GET `/apps/:id/anrGroups/:anrGroupId/plots/instances` API to
support no app versions
- Modify GET `/apps/:id/anrGroups/:anrGroupId/plots/distribution` API to
support no app versions

fixes #1725

Signed-off-by: detj <[email protected]>
@detj detj added bug something isn't working backend backend related labels Jan 16, 2025
@detj detj self-assigned this Jan 16, 2025
Copy link

vercel bot commented Jan 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
measure-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 16, 2025 6:29am

- Modify GET `/apps/:id/sessions` API to support no app versions

Signed-off-by: detj <[email protected]>
@detj
Copy link
Contributor Author

detj commented Jan 20, 2025

Closing this PR as we won't support "none" option in app version selector, while other selectors will gain "none" support separately.

@detj detj closed this Jan 20, 2025
@detj detj deleted the selected-versions-none branch January 20, 2025 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend backend related bug something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Make APIs work with no app version filters
1 participant