stac-layer icon indicating copy to clipboard operation
stac-layer copied to clipboard

Pass bands that are displayed via event

Open DanielJDufour opened this issue 3 years ago • 4 comments

http://localhost:8080/test/stac-item/band-info.html test passes, but when I click on the lyr, the callback data doesn't appear to include bands.

DanielJDufour avatar Jun 09 '22 14:06 DanielJDufour

Was this already the case in 0.11.0 with Leaflet 1.7?

m-mohr avatar Jun 09 '22 14:06 m-mohr

Yes, I assume so.

DanielJDufour avatar Jun 09 '22 14:06 DanielJDufour

I checked the implementation and currently stac-layer just passes the eo:bands completely and not the bands that are actually shown/used. As such the implementation can be removed and replaced with stac-js functionality to retrieve bands. Long-term we should actually return the bands used, but that's not even the case right now.

m-mohr avatar Mar 04 '23 01:03 m-mohr

Sounds like a good plan

DanielJDufour avatar Mar 04 '23 14:03 DanielJDufour