Skip to content

feat(heremap): allow choosing map engine via prop#19

Merged
KhaledSakr merged 2 commits intomasterfrom
feature/pos-2126
Feb 20, 2025
Merged

feat(heremap): allow choosing map engine via prop#19
KhaledSakr merged 2 commits intomasterfrom
feature/pos-2126

Conversation

@KhaledSakr
Copy link
Member

@KhaledSakr KhaledSakr commented Feb 12, 2025

What the PR Includes

  • allow choosing map engine via prop.

Checklist

  • Tests are added.
  • Manual testing instructions are added.
  • Configs are added/adapted if applicable.
  • Issue is properly linked if applicable.

Manual Testing Instructions

  1. Can be tested along with https://github.com/impargo/impargo-apps/pull/9705

@KhaledSakr
Copy link
Member Author

/publish

@david-paulus-saad
Copy link
Collaborator

RUN URL: https://github.com/impargo/react-here-maps/actions/runs/13287687924
PACKAGE VERSION: 3.0.0-pre-28

@KhaledSakr
Copy link
Member Author

/publish

@david-paulus-saad
Copy link
Collaborator

RUN URL: https://github.com/impargo/react-here-maps/actions/runs/13290063314
PACKAGE VERSION: 3.0.0-pre-29

@KhaledSakr
Copy link
Member Author

/publish

@david-paulus-saad
Copy link
Collaborator

RUN URL: https://github.com/impargo/react-here-maps/actions/runs/13292372282
PACKAGE VERSION: 3.0.0-pre-33

defaultLayers?: DefaultLayers,
apiKey: string,
useVectorTiles: boolean,
enable: boolean,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess the name is a bit vague

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense. I changed it to enableVectorLayers and enableRasterLayers

@KhaledSakr KhaledSakr merged commit 3ffe8f1 into master Feb 20, 2025
6 checks passed
@KhaledSakr KhaledSakr deleted the feature/pos-2126 branch February 20, 2025 09:25
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.

3 participants