maps icon indicating copy to clipboard operation
maps copied to clipboard

Clarify Expo plugin configuration instructions

Open pwellner opened this issue 2 years ago • 6 comments

Description

Tiny clarification to Expo plugin doc

Checklist

  • [x ] I've read CONTRIBUTING.md

pwellner avatar Feb 19 '24 19:02 pwellner

@pwellner download and access token are different also install instructions are in our docs site. https://rnmapbox.github.io/docs/install

mfazekas avatar Feb 19 '24 19:02 mfazekas

Thanks @mfazekas ! I was a bit confused by these instructions.

Do you think this proposed change makes things more clear to someone just starting out?

pwellner avatar Feb 19 '24 21:02 pwellner

Thanks @mfazekas ! I was a bit confused by these instructions.

Do you think this proposed change makes things more clear to someone just starting out?

Thanks for suggesting these changes. I was trying to figure this out right now and fortunately found your PR.

I personally find this to be clearer, but the issue I ran into is actually discovery.

On the Installation instruction page there is no example of the actual usage of Mapbox.setAccessToken, there is only a mention at the top: "public token to use as accessToken when running the app".

In the examples on the website, the map initialization using setAccessToken is also not mentioned. It isn't even a result in the search function.

MoreSaltMoreLemon avatar Feb 20 '24 03:02 MoreSaltMoreLemon

Thanks @mfazekas ! I was a bit confused by these instructions.

Do you think this proposed change makes things more clear to someone just starting out?

Thanks @mfazekas ! I was a bit confused by these instructions. Do you think this proposed change makes things more clear to someone just starting out?

Thanks for suggesting these changes. I was trying to figure this out right now and fortunately found your PR.

I personally find this to be clearer, but the issue I ran into is actually discovery.

On the Installation instruction page there is no example of the actual usage of Mapbox.setAccessToken, there is only a mention at the top: "public token to use as accessToken when running the app".

In the examples on the website, the map initialization using setAccessToken is also not mentioned. It isn't even a result in the search function.

Great point, we'd need an Adding a Map section where we show how to call setAccessToken and how to add a trivial MapView component.

mfazekas avatar Feb 20 '24 06:02 mfazekas

Where should the changes be made?

In the maps repo here? https://github.com/pwellner/maps/blob/main/plugin/install.md

Or in the maps-docs repo here? https://github.com/pwellner/maps-docs/blob/main/docs/install.md

pwellner avatar Feb 20 '24 15:02 pwellner

@pwellner I'd like to focus the doc site more and make the repo docs just refer to the doc site.

mfazekas avatar Feb 20 '24 17:02 mfazekas