elastic/kibana

[Field Formats] Duration formatter should handle pluralization for single values

Open

#196 761 ouverte le 17 oct. 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)TypeScript (8 021 forks)batch import
Feature:FieldFormattersTeam:DataDiscoverygood first issueimpact:lowloe:small

Métriques du dépôt

Stars
 (19 065 stars)
Métriques de merge PR
 (Merge moyen 1j 16h) (999 PRs mergées en 30 j)

Description

Describe the feature:

The Friendly duration formatters should account for singular values. Currently, we only use pluralized formats.

This is where we compute the unit text and join it with the value...

https://github.com/elastic/kibana/blob/89d833bf58ba614a0e9eb6a38a02c4eb5a641aeb/src/plugins/field_formats/common/converters/duration.ts#L134-L137

Here is the list of all values...

https://github.com/elastic/kibana/blob/89d833bf58ba614a0e9eb6a38a02c4eb5a641aeb/src/plugins/field_formats/common/constants/duration_formats.ts#L18-L167

This only applies to the long form text strings and not the shortText strings.

Describe a specific use case for the feature:

Image

Notice 1 seconds should be 1 second.

Pretty simple and low impact but could be nice.

Guide contributeur

Direction de recherche
Le problème concerne la pluralisation dans le formateur Friendly duration. Le code pertinent se trouve dans src/plugins/field formats/common/converters/duration.ts aux lignes 134-137. Actuellement, la fonction utilise la forme plurielle de la chaîne d'unité pour toutes les valeurs. Pour corriger, vérifiez si la valeur est exactement 1, et si c'est le cas, utilisez la forme singulière du fichier de constantes dans src/plugins/field formats/common/constants/duration formats.ts. Le changement n'affecte que le format 'text' (long), pas 'shortText'. Implémentez la condition et assurez vous que les tests passent.
Stack technique
typescript
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85