Scalars lose their directives, including `@specifiedBy`
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #894 di @oryan-block — integrata
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 55/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- graphql, kotlin
- Ambito
- backend-api-design
Direzione di ricerca
Start by reading how SchemaClassScanner builds scalars from values passed to .scalars(...), then compare that with how SchemaParser builds other types and applies directivesOf and SchemaDirectiveWiring. Check scalar definitions and extensions, including the provided scalar's specifiedByUrl and directives. Done means SDL and supplied-scalar directives and specifiedByUrl are preserved in the printed schema.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Scalars are built in SchemaClassScanner from the GraphQLScalarType passed to .scalars(...), but only its name, description and coercing are copied. Directives from the SDL are never applied, on the scalar or on its extensions:
scalar Date @specifiedBy(url: "https://example.com/date")
prints as scalar Date. The provided scalar's own specifiedByUrl and directives aren't copied either.
Probably easiest to build scalars in SchemaParser like the other types, so they go through directivesOf and SchemaDirectiveWiring too.
- Lingua principale
- Kotlin
- Stelle
- 824
- Fork
- 175
- Merge medio
- 2g 1h
- PR unite (30g)
- 59
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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-java-kickstart/graphql-java-tools
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
graphql-java-kickstart/graphql-java-tools#893 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
graphql-java-kickstart/graphql-java-tools#905 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
graphql-java-kickstart/graphql-java-tools#900 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
I maintainer di solito rispondono entro 1 giorno
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Apertaenhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di graphql-java-kickstart/graphql-java-tools
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
mcnaveen/health-connect-webhook#110 ·
I maintainer di solito rispondono entro 1 giorno
-
Delete all aboveApertaenhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
mjaakko/NeoStumbler#1307 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
No Wallpaper: Black and white theme changes makes fonts look weird on Home screen and App listAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
tanujnotes/Olauncher#770 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
-
[Submission] PayPalApertasubmit-adaption submit-adaption-pre
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 76/100
BetterAndroid/android-notification-icon-project#795 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno