vuejs/vue-router

The type definition of Route seems to be partially wrong

Aberta

#3.566 aberto em 9 de jun. de 2021

Ver no GitHub
 (4 comentários) (0 reação) (0 responsável)JavaScript (5.092 forks)batch import
Typescriptcontribution welcomefixed on 4.xgood first issue

Métricas do repositório

Stars
 (19.015 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Version

3.5.1

Reproduction link

https://github.com/vuejs/vue-router/blob/dev/types/router.d.ts#L203

Steps to reproduce

None, just the source code seems to be wrong.

What is expected?

The type definition of Route#query should be

query: Dictionary<string | null | (string | null)[]>

The current implementation cannot express this type of query

/path?foo#bar

as it doesn't have any value and it is not converted to an array.

What is actually happening?

https://github.com/vuejs/vue-router/blob/dev/types/router.d.ts#L203

The type definition of Route#query is

query: Dictionary<string | (string | null)[]>

Guia do colaborador

Direção de pesquisa
Revise a definição de tipo em types/router.d.ts e entenda o tipo query da Route. Confirme a assinatura de tipo esperada e garanta consistência com a manipulação de valores de consulta no código.
Pilha de tecnologia
typescriptvue
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
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
TypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75