downshift-js/downshift

TypeScript: have items and selectedItems as Readonly

Offen

#1.030 geöffnet am 30.04.2020

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (980 Forks)batch import
TypeScripthelp wanted

Repository-Metriken

Stars
 (11.761 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

  • 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.

Contributor Guide

Research-Richtung
Öffnen Sie die Datei typings.ts für useSelect, useCombobox und useMultipleSelection und umschließen Sie die Typen items und selectedItems mit Readonly.
Tech Stack
typescriptreactjavascript
Domain
frontend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptReact
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85