Skip to content

Conversation

@mrMetalWood
Copy link
Contributor

@mrMetalWood mrMetalWood requested review from andreashelms and ro-ka July 7, 2020 07:43
README.md Outdated
mapContainer={mapContainer}
options={mapOptions}
libraries={['places']}
mapIds={['d85da84f141b8b84']}
Copy link
Contributor

Choose a reason for hiding this comment

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

Where does this id come from?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a random test id that I created. There will be no actual mapstyle with this id. I just wanted to include an authentic id to show what an id looks like.

You have to create your ids in the cloud console.

Copy link
Contributor

Choose a reason for hiding this comment

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

I would rather use your-mapstyle-id or something, otherwise it's not clear whether this id is valid or not. But only as suggestion, not blocking here.

andreashelms
andreashelms previously approved these changes Jul 7, 2020
@mrMetalWood mrMetalWood merged commit 18b8265 into master Jul 7, 2020
@mrMetalWood mrMetalWood deleted the feat/map-ids branch July 7, 2020 09:40
mapContainer={mapContainer}
options={mapOptions}
libraries={['places']}
mapIds={['<your-mapstyle-id>']}
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

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