-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[ML] AIOps: Enhance the UX for grouped results which only contain field / value pairs that appear in other groups #148757
[ML] AIOps: Enhance the UX for grouped results which only contain field / value pairs that appear in other groups #148757
Conversation
Pinging @elastic/ml-ui (:ml) |
{repeatedValues.length > 0 && !valuesBadges.length ? ( | ||
<FormattedMessage | ||
id="xpack.aiops.explainLogRateSpikes.spikeAnalysisTableGroups.onlyMoreRepeatedLabel" | ||
defaultMessage="{count, plural, one {# field/value pair} other {# more field/value pairs}} also appearing in other groups" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to drop the more
from the other
case too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yes - don't know how I missed that 🙏 - added to the singular case in cf37b20
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary
Related meta issue: #146162
This PR updates the UX for grouped results which only contain field / value pairs that appear in other groups. In the updated text the '+' and 'more' are dropped from the label.
Checklist
Delete any items that are not applicable to this PR.