Graylog2/graylog2-server

Don't sort output config fields by optionality

Open

#4761 aperta il 25 apr 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Java (1032 fork)batch import
good first issuepapercuttriaged

Metriche repository

Star
 (6945 star)
Metriche merge PR
 (Merge medio 8g 19h) (297 PR mergiate in 30 g)

Descrizione

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)

Guida contributor

Direzione di ricerca
Apri il file ConfigurationForm.jsx, individua la riga 156 e rimuovi o modifica la funzione di ordinamento che ordina i campi per opzionalità.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90