downshift-js/downshift

TypeScript: have items and selectedItems as Readonly

Aperta

#1030 aperta il 30 apr 2020

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

Metriche repository

Star
 (11.761 stelle)
Metriche merge PR
 (Merge medio 6g 9h) (5 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
Apri il file typings.ts per useSelect, useCombobox e useMultipleSelection, quindi racchiudi i tipi items e selectedItems con Readonly.
Tech stack
typescriptreactjavascript
Dominio
frontend
Tipo issue
Refactoring
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85