mapbox / mapbox/mapbox-gl-geocoder
Update debug pages to be... real pages
Open
Nobody has claimed this yet.
auto-triaged
enhancement
- Dominant language
- JavaScript
- Stars
- 388
- Forks
- 181
- PR merge metrics
- No merged PRs in 30d
Description
Per conversation with @scottsfarley93, the current budo javascript-only format is strange and doesn't provide an intuitive example of how to use the geocoder.
Also, setting the access token using localStorage is awkward. We should standardize between the tests (which use env vars) and here.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect the current debug pages and the tests that use environment variables; compare them with the budo JavaScript-only format and localStorage token handling. Done means the debug pages are real pages, provide an intuitive geocoder example, and use the same access-token configuration convention as the tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend, web-dev
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100