react-component / react-component/table
Use DataIndex<RecordType> for better constraints?
Nessuno ha ancora preso questa issue.
- Lingua principale
- TypeScript
- Stelle
- 1.4k
- Fork
- 618
- Merge medio
- 10h 19m
- PR unite (30g)
- 2
Descrizione
I'd like to propose this change:
Before:
export declare type DataIndex = string | number | (string | number)[];
After:
export declare type DataIndex<RecordType> = keyof RecordType | number | (keyof RecordType | number)[];
That way, you get full type checking on dataIndex like here:
const columns : ColumnsType<Client>= [
{
title: 'Name',
dataIndex: 'name', // If 'name' does not exist for type Client, a Typescript error will occur
key: 'name',
render: text => <a>{text}</a>
},
]
If you like it, I can send a PR.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia individuando le dichiarazioni dei tipi DataIndex e ColumnsType e verifica come viene usato dataIndex nei tipi della tabella. Confronta il vincolo generico proposto con gli usi esistenti, quindi verifica che le chiavi valide superino il controllo dei tipi e che le chiavi non valide producano errori TypeScript senza compromettere gli indici numerici o di array.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- react, typescript
- Ambito
- frontend
- Tipo di issue
- Funzionalità
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100