Skip to content

Conversation

@devotaaabel
Copy link
Contributor

This PR adds support for POST requests to the directions API.

Closes #957

* Use POST method to request data.
* The default is to use GET.
* @return this builder for chaining options together
* @since 4.4.0
Copy link
Contributor

Choose a reason for hiding this comment

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

should it be 4.6.0?

/**
* Use GET method to request data.
* @return this builder for chaining options together
* @since 4.4.0
Copy link
Contributor

Choose a reason for hiding this comment

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

should it be 4.6.0?


@Test
public void testPost() throws IOException {
MapboxDirections mapboxDirections = MapboxDirections.builder()
Copy link
Contributor

Choose a reason for hiding this comment

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

Should you add a post fixture to avoid the bug that you found in MapboxMapMatching?

@devotaaabel devotaaabel merged commit df82399 into master Feb 18, 2019
@devotaaabel devotaaabel deleted the devota-directions-post branch February 18, 2019 22:43
@osana osana mentioned this pull request Apr 11, 2019
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants