required should be inferred from the return type of a hybrid property
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 52/100
Direzione di ricerca
Inizia tracciando come i campi hybrid_property vengono convertiti in campi GraphQL, quindi confronta questo percorso con l’inferenza esistente della nullabilità delle proprietà delle colonne. La modifica è completata quando un tipo restituito bool dichiarato produce un campo Boolean obbligatorio e bool | None ne produce uno facoltativo, con una copertura per entrambi i casi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
With a definition like:
@hybrid_property
def is_active(self) -> bool:
return self.state == 'ACTIVE'
The resulting graphql field type is currently marked Boolean unless overridden by ORMField(). Column properties correctly determine the nullability from the column definition. It seems reasonable to do the same for the hybrid_property based on the declared type. If the return type were bool | None, then it would be optional, otherwise it should be required.
- Lingua principale
- Python
- Stelle
- 985
- Fork
- 224
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Apri 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 graphql-python/graphene-sqlalchemy
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
graphql-python/graphene-sqlalchemy#422 · 1 commento ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
graphql-python/graphene-sqlalchemy#419 · 3 commenti ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
graphql-python/graphene-sqlalchemy#418 · 2 commenti · 2 reazioni ·
Tutte le issue di graphql-python/graphene-sqlalchemy
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
anthropics/skills#1811 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
speaches-ai/speaches#678 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
datalayer/mcp-compose#42 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
conda-forge/spacy-feedstock#177 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
UKGovernmentBEIS/inspect_evals#2523 ·