mapbox / mapbox/mapbox-maps-flutter
Leverage Turf for Geometry in TileRegionLoadOptions
Nessuno ha ancora preso questa issue.
- Lingua principale
- Dart
- Stelle
- 380
- Fork
- 204
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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?` ?
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.
Direzione di ricerca
Inizia in lib/src/offline/offline_messenger.dart intorno alla riga 348 e ispeziona l’API TileRegionLoadOptions generata. Confronta i tipi di geometry e descriptorOptions con l’utilizzo corrispondente dell’API basata su Turf. Il lavoro è completato quando i tipi previsti e la nullabilità sono coerenti nell’interfaccia generata, anche se l’issue segnala che le definizioni sorgente non sono pubbliche.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- dart, flutter
- Ambito
- api, mobile
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 25/100