Skip to content

Add map-type-hybrid sample#112

Merged
manz merged 4 commits intomasterfrom
feat/map-type-hybrid
Mar 5, 2026
Merged

Add map-type-hybrid sample#112
manz merged 4 commits intomasterfrom
feat/map-type-hybrid

Conversation

@manz
Copy link
Member

@manz manz commented Mar 5, 2026

Issue

Companion to Woosmap/maps-js#1494 which adds mapTypeId support (ROADMAP & HYBRID) to the Map JS SDK.

Describe your changes

New sample that initializes a map with hybrid satellite view centered on the Eiffel Tower and listens for mapTypeId_changed events when the user toggles the built-in Map / Satellite control.

Uses @ts-ignore for mapTypeId, MapTypeId, and getMapTypeId until @types/woosmap.map is updated for Map JS API v1.33.

Will be embedded in the developers.woosmap.com Map Types concept page (PR #1246).

How to test

PR preview will be available at https://demo.woosmap.com/misc/js-samples-pr-deploy/pr-{N}/samples/map-type-hybrid/app/dist/ once CI deploys to S3.

Locally: npm start then open http://localhost:8080/samples/map-type-hybrid/app/dist/

Checklist:

  • My code follows the style guidelines for this repo
  • My code passes the SonarCloud check and does not add new code smells
  • I have performed a self-review of my code
  • My changes generate no new warnings/errors
  • I have added tests that prove my fix is effective or that my feature works
  • I don't require ops changes for this PR to go to prod
  • This change does not include a migration

manz added 3 commits March 5, 2026 11:13
Demonstrates hybrid (satellite) map initialization with mapTypeId
and the built-in map type control for switching between roadmap and satellite.
gaelsimon
gaelsimon previously approved these changes Mar 5, 2026
@manz manz self-assigned this Mar 5, 2026
@manz
Copy link
Member Author

manz commented Mar 5, 2026

I'll probably revisit this PR when We'll be able to publish typedefs again

@manz manz merged commit 87bceaf into master Mar 5, 2026
10 of 29 checks passed
@manz manz deleted the feat/map-type-hybrid branch March 5, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants