mapbox / mapbox.js

Mapbox JavaScript API, a Leaflet Plugin
mapbox.com/mapbox.js/
Other
1.91k stars 386 forks source link

Final Audit of Examples for Classic Styles #1345

Closed riastrad closed 3 years ago

riastrad commented 3 years ago

Context

Support for Mapbox Classic Styles has been turned off (ref. this blog post). In order to make sure that Mapbox.js repos continue to function as expected, this PR reflects a final audit of the documentation's use of these resources.

The results of this audit:

  1. Removal of 10 examples that rely on deprecated APIs
  2. Updating 7 examples to no longer rely on deprecated classic styles
riastrad commented 3 years ago

Thanks for the quick review, @katydecorah.

do you think any of these need redirects?

All of the deleted examples are being removed because they're built on features of the Legacy Maps API that are no longer available. So no redirects feels appropriate. And in the event that the library is updated in such a way that this functionality is possible again without the API, we can always add back versions of these examples.