Remove deprecated missingResolverDataFetcher option in 15.0.0
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 65/100
- Issue-Typ
- Refactoring
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- graphql, kotlin
- Bereich
- backend-api-design
Rechercherichtung
Read SchemaParserOptions, MissingFieldResolver, and FieldResolverScanner to trace the deprecated option through its constructor, builder, and fallback checks. Then inspect MissingFieldResolverTest and remove the test for that option. Done means those paths no longer use missingResolverDataFetcher while missingResolverDataFetcherProvider remains supported; the issue also lists two other breaking cleanups for 15.0.0.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
SchemaParserOptions.missingResolverDataFetcher has been deprecated since 13.0.4 (8b27ed6, March 2023) in favour of missingResolverDataFetcherProvider. We should remove it in the next major release (15.0.0).
It's a breaking change, so it can't go into a 14.x release: SchemaParserOptions.Builder.missingResolverDataFetcher(...) and the SchemaParserOptions.missingResolverDataFetcher property are both public.
What to remove
missingResolverDataFetcherfrom theSchemaParserOptionsconstructor and the builder- the fallback to it in
MissingFieldResolverand the null check inFieldResolverScanner.missingFieldResolver - the test in
MissingFieldResolverTestthat covers the deprecated option
Migration
// before
.missingResolverDataFetcher(fetcher)
// after
.missingResolverDataFetcherProvider { _, _ -> fetcher }
Other breaking cleanups for the same release
- the
environment.getContext()fallback for custom context classes inMethodFieldResolver(already has a "remove in next major" TODO) - the
getContext()fallback for aCoroutineScopepassed as the legacy context object inUtils.coroutineScope(); it now logs a warning and should be read fromGraphQLContextinstead
- Vorherrschende Sprache
- Kotlin
- Sterne
- 824
- Forks
- 175
- Ø Merge
- 2 T. 5 Std.
- Gemergte PRs (30 T.)
- 53
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus graphql-java-kickstart/graphql-java-tools
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
graphql-java-kickstart/graphql-java-tools#893 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 74/100
graphql-java-kickstart/graphql-java-tools#896 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
graphql-java-kickstart/graphql-java-tools#886 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Offenenhancement
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in graphql-java-kickstart/graphql-java-tools
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
leboncoin/spark-android#2342 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Bug Domain changed
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
keiyoushi/extensions-source#20257 · 1 Kommentar · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag
-
feature
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
user-grinch/RivoPhoneApp#302 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
LeanBitLab/LeanType#713 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag