Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fixed multiple paths #149

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fixed multiple paths #149

wants to merge 1 commit into from

Commits on Nov 4, 2017

  1. fixed multiple paths 

    When parsing the path array each next path must be transformed to &path=
The first path is transformed correctly but each next path is not.
Removing the .join('|') will fix this bug.

See https://developers.google.com/maps/documentation/static-maps/intro#URL_Parameters
-> Feature Parameters -> path:

"path (optional) defines a single path of two or more connected points to overlay on the image at specified locations. This parameter takes a string of point definitions separated by the pipe character (|). You may supply additional paths by adding additional path parameters. Note that if you supply a path for a map, you do not need to specify the (normally required) center and zoom parameters. For more information, see Google Static Maps API Paths below."

-> "You may supply additional paths by adding additional path parameters." and so each URL parameter "path" has the following format:

 &path=
    @CharlyGeen
    CharlyGeen authored Nov 4, 2017
    Configuration menu
    Copy the full SHA
    8f4c2bb View commit details
    Browse the repository at this point in the history