React-query admin wrapper and possibily undefined value with useGetOne
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- react, typescript
- Ambito
- frontend
Direzione di ricerca
Inizia da packages/ra-core/src/dataProvider/useGetOne.ts e dall'identifier type in packages/ra-core/src/types.ts, quindi confronta i relativi tipi con l'opzione enabled di React Query. Verifica che l'hook possa rappresentare un id undefined quando la query è disabilitata e verifica il comportamento dei tipi risultante con i type check o i test pertinenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
According to the React Query documentation, we have the possibility to block the running query thanks to the enabled function if one of our parameter is not defined.
On the documentation sample, we can see userId can be set on the query definition without any issue.
We also have the possibility to use this option from the useGetOne hooks as you specified on the code doc block
However, the typing of the function requires the id to be a valid identifier:
Because of that, we have to cheat by casting the typing of the undefined value, for example:
const { data: userData } = useGetOne<UserRecord>('users', { id: userId || '' }, { enabled: !!userId });
According to me, we should make that param optional to match the react-query implementation.
What do you think?
- Lingua principale
- TypeScript
- Stelle
- 27k
- Fork
- 5.5k
- Merge medio
- 1g 2h
- PR unite (30g)
- 32
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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.
Altre issue di marmelab/react-admin
-
Add emptyText property to SelectArrayInputForse già presa @kwitykwity l’ha presa 63 giorni fa. Apertaenhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
marmelab/react-admin#10454 · 7 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 Più di una settimana Idoneità per principianti 38/100
marmelab/react-admin#11426 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
ra-data-graphql-simple: outdated graphql-ast-types-browser dependency causes npm peer warningsForse di nuovo libera @bipul724 l’ha presa 16 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
marmelab/react-admin#11395 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Fix decode-uri-component DoS (CVE-2026-45822) by dropping query-stringForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
marmelab/react-admin#11380 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
useCreate > onError returns callback function where error is "unknown"Forse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
marmelab/react-admin#11262 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di marmelab/react-admin
Issue simili
-
🐛 bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
LibreChat-AI/LibreChat#17733 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
TypeStrong/typedoc#3150 ·
-
accessibility bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
Normalize image-source MIME types before SVG dispatchForse già presa @HarsH-1877 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
tscircuit/image-utils#58 · 1 commento ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 74/100
I maintainer di solito rispondono entro 2 giorni