vuejs/vue-router

The type definition of Route seems to be partially wrong

Aperta

#3566 aperta il 9 giu 2021

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (5092 fork)batch import
Typescriptcontribution welcomefixed on 4.xgood first issue

Metriche repository

Star
 (19.015 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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)[]>

Guida contributor

Direzione di ricerca
Rivedi la definizione del tipo in types/router.d.ts e comprendi il tipo query di Route. Conferma la firma del tipo prevista e assicurati che sia coerente con la gestione dei valori di query nel codebase.
Tech stack
typescriptvue
Dominio
frontend
Tipo issue
Bug
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75