mapbox / mapbox/mapbox-maps-flutter
Leverage Turf for Geometry in TileRegionLoadOptions
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Dart
- Estrellas
- 380
- Forks
- 204
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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?` ?
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza en lib/src/offline/offline_messenger.dart alrededor de la línea 348 e inspecciona la API generada de TileRegionLoadOptions. Compara los tipos de geometry y descriptorOptions con el uso correspondiente de la API basada en Turf. Se considera terminado cuando los tipos previstos y la nulabilidad son coherentes en la interfaz generada, aunque el issue señala que las definiciones de origen no son públicas.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- dart, flutter
- Área
- api, mobile
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 25/100