fix: add promoteId and properties.id to mapbox-gl.adapter.ts

JamesLMilner / terra-draw

A library for drawing on maps that supports Mapbox, MapLibre, Google Maps, OpenLayers and Leaflet out the box

https://terradraw.io

MIT License

447 stars 50 forks source link

fix: add promoteId and properties.id to mapbox-gl.adapter.ts #267

Closed ScottEAdams closed 1 week ago

ScottEAdams commented 2 months ago

Description of Changes

Added promoteId to _addGeoJSONSource and properties.id in render.

Link to Issue

https://github.com/JamesLMilner/terra-draw/issues/266

PR Checklist

[ x ] There is a associated GitHub issue
[ n/a ] If I have added significant code changes, there are relevant tests
[ n/a ] If there are behaviour changes these are documented

JamesLMilner commented 1 week ago

As discussed in https://github.com/JamesLMilner/terra-draw/issues/266 this is mostly to support custom requirements which are probably outside the scope of the default TerraDrawMapboxAdapter. Open to further discussion on this topic if other people feel they would benefit from this for other reasons however!