mapbox / mapbox/mapbox-java

Add routing support to MapboxGeocoding

Open
#1,049 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Geocoding
Dominant language
Java
Stars
438
Forks
117
PR merge metrics
No merged PRs in 30d

Description

Expected Behavior

The Java SDK's MapboxGeocoding.builder() should be able to take a routing boolean.

https://docs.mapbox.com/api/search/#forward-geocoding

Screen Shot 2019-06-14 at 9 14 45 AM
Current Behavior

There's currently no builder method for this geocoding parameter.

Possible Solution

add a routing(boolean) method to MapboxGeocoding.builder() and routing setup to the GeocodingService class

Context (Environment)

Mismatch between current Geocoding API offerings and the Java SDK's capabilities were brought up amongst some search chat between @yuletide , @scottsfarley93 , and others.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at MapboxGeocoding.builder() and trace how its options reach the GeocodingService class. Follow the existing geocoding parameter setup, then verify that a routing boolean can be supplied through the builder and is included in the request sent to the Mapbox Geocoding API.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.