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

Address multi value aggregation keys for reporting #1895

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

Fajfa
Copy link
Member

@Fajfa Fajfa commented Aug 28, 2024

The following changes are implemented

TODO: Summary

Changes in the user interface:

TODO: Add screenshots, recordings or remove this section

Checklist when submitting a final (!draft) PR

  • Commits are tidied up, squashed if needed and follow guidelines in CONTRIBUTING.md
  • Code builds
  • All existing tests pass
  • All new critical code is covered by tests
  • PR is linked to the relevant issue(s)
  • Rebased with the target branch

@Fajfa Fajfa linked an issue Aug 28, 2024 that may be closed by this pull request
1 task
@Fajfa Fajfa force-pushed the 2023.9.x-fix-chart-mv-keys branch from fc90626 to b43da9f Compare August 29, 2024 10:48
@Fajfa Fajfa merged commit b43da9f into 2023.9.x Aug 29, 2024
11 checks passed
@Fajfa Fajfa deleted the 2023.9.x-fix-chart-mv-keys branch August 29, 2024 10:48
Copy link

sentry-io bot commented Sep 9, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ runtime.boundsError: runtime error: index out of range [-1] GET /api/compose/namespace/402510356586037249/m... View Issue

Did you find this useful? React with a 👍 or 👎

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.

Multiple values are ignored in charts. Only the first value is counted.
2 participants