mapbox / mapbox/mapbox-maps-flutter
Leverage Turf for Geometry in TileRegionLoadOptions
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Dart
- Sterne
- 380
- Forks
- 204
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Most API use the Turf classes, however `TileRegionLoadOptions` has a field `geometry` which is of type `Map?` instead of `turf.GeometryType?`. Since this is defined in a pigeon generated file and the sources from which those files are generated are not public I'm unfortunately not able to submit a PR. @maios Is there any reason why the field does not use `turf.GeometryType` or was this just forgotten?
https://github.com/mapbox/mapbox-maps-flutter/blob/e688ca96c76c0c58d4dcb7c723bdaf625e41f6b8/lib/src/offline/offline_messenger.dart#L348
Also why is `descriptorOptions` of type `List?` and not `List?` ?
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in lib/src/offline/offline_messenger.dart ungefähr bei Zeile 348 und untersuche die generierte TileRegionLoadOptions API. Vergleiche die Typen von geometry und descriptorOptions mit der entsprechenden Turf-basierten API-Verwendung. Die Aufgabe ist abgeschlossen, wenn die vorgesehenen Typen und die Nullbarkeit in der generierten Schnittstelle konsistent sind, wobei der Issue anmerkt, dass die Quelldefinitionen nicht öffentlich sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- dart, flutter
- Bereich
- api, mobile
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 25/100