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 [Field Formats] Duration formatter should handle pluralization for single values elastic/kibana#196761 #196987

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ZakharA
Copy link

@ZakharA ZakharA commented Oct 20, 2024

Summary

Summarize your PR. If it involves visual changes include a screenshot or gif.
No visual changes were made, and unit tests were updated.
Not sure where I should put it here but all the tests are passing
image

Checklist

Delete any items that are not applicable to this PR.

@ZakharA ZakharA requested a review from a team as a code owner October 20, 2024 19:20
Copy link

cla-checker-service bot commented Oct 20, 2024

❌ Author of the following commits did not sign a Contributor Agreement:
94106df, d654232

Please, read and sign the above mentioned agreement if you want to contribute to this project

@ZakharA ZakharA changed the title handle pluralization for single values Fixes https://github.com/elastic/kibana/issues/196761 Oct 20, 2024
@ZakharA ZakharA changed the title Fixes https://github.com/elastic/kibana/issues/196761 Fixes elastic/kibana#196761 Oct 20, 2024
@ZakharA ZakharA changed the title Fixes elastic/kibana#196761 fix elastic/kibana#196761 [Field Formats] Duration formatter should handle pluralization for single values Oct 20, 2024
@ZakharA ZakharA changed the title fix elastic/kibana#196761 [Field Formats] Duration formatter should handle pluralization for single values fix [Field Formats] Duration formatter should handle pluralization for single values elastic/kibana#196761 Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants