Skip to content

Conversation

@suzit-10
Copy link
Contributor

@suzit-10 suzit-10 commented Sep 23, 2025

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Related Issue

Describe this PR

This PR removes all Mapbox-specific code paths and conditions that relied on a Mapbox token or Mapbox-only styles.

Changes

  • Removed checks for Mapbox token availability.
  • Dropped Mapbox-specific style rendering.
  • Updated logic to consistently use MapLibre code paths.

Important Note:

We are still using some Mapbox packages that provide features on top of MapLibre:

  • @mapbox/mapbox-gl-draw β†’ drawing tools
  • @mapbox/mapbox-gl-language β†’ language switching support

Why these are kept

  • Both libraries are open-source with permissive licenses (ISC for Draw, BSD-3 for Language).
  • They do not require a Mapbox token or Mapbox services.
  • They enhance map functionality (drawing and language switching) while maintaining compatibility with MapLibre.

Eliminates checks for Mapbox tokens.

Eliminates checks for Mapbox tokens and render Mapbox-only style.
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
C Maintainability Rating on New Code (required β‰₯ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants