downshift-js/downshift

TypeScript: have items and selectedItems as Readonly

Open

#1030 aperta il 30 apr 2020

Vedi su GitHub
 (4 commenti) (1 reazione) (0 assegnatari)JavaScript (980 fork)batch import
TypeScripthelp wanted

Metriche repository

Star
 (11.761 star)
Metriche merge PR
 (Merge medio 4g 9h) (7 PR mergiate in 30 g)

Descrizione

  • downshift version:
  • node version:
  • npm (or yarn) version:

Relevant code or config

What you did:

What happened:

Reproduction repository:

Problem description: Since we use items/selectedItems just for some logic and we do not tamper with them, we can mark them as read only in the TS interface.

Suggested solution: In typings.ts for useSelect, useCombobox and useMultipleSelection, use Readonly to wrap the items type.

Guida contributor

Direzione di ricerca
Individua il file typings.ts nel codice sorgente di downshift. Per gli hooks useSelect, useCombobox e useMultipleSelection, modifica le definizioni dei tipi per avvolgere le proprietà items e selectedItems con Readonly<T> per indicare che non sono mutabili. Assicura la coerenza tra tutti e tre gli hooks. Non sono richieste modifiche alla logica.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
Basic TypeScript knowledgeFamiliarity with downshift typings
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90