Graylog2/graylog2-server

Don't sort output config fields by optionality

Open

#4 761 ouverte le 25 avr. 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Java (1 032 forks)batch import
good first issuepapercuttriaged

Métriques du dépôt

Stars
 (6 945 stars)
Métriques de merge PR
 (Merge moyen 8j 19h) (297 PRs mergées en 30 j)

Description

Expected Behavior

The config fields of my output plugin appear in the order I registered them.

Current Behavior

If some of them are optional, they are shown below all others.

Possible Solution

Don't sort the fields by optionality.

Steps to Reproduce (for bugs)

Register two output config fields and make the first optional.

Context

https://github.com/Graylog2/graylog2-server/pull/4746#issuecomment-384253076

Your Environment

(not relevant)

Guide contributeur

Direction de recherche
Ouvrez le fichier ConfigurationForm.jsx, localisez la ligne 156 et supprimez ou modifiez la fonction de tri qui trie les champs par optionalité.
Stack technique
javascriptreact
Domaine
frontend
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.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90