downshift-js/downshift

TypeScript: have items and selectedItems as Readonly

Open

#1.030 aberto em 30 de abr. de 2020

Ver no GitHub
 (4 comments) (1 reaction) (0 assignees)JavaScript (980 forks)batch import
TypeScripthelp wanted

Métricas do repositório

Stars
 (11.761 stars)
Métricas de merge de PR
 (Mesclagem média 6d 9h) (5 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Abra o arquivo typings.ts para useSelect, useCombobox e useMultipleSelection e envolva os tipos items e selectedItems com Readonly.
Pilha de tecnologia
typescriptreactjavascript
Domain
frontend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScriptReact
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85