Skip to content
This repository has been archived by the owner on Apr 24, 2024. It is now read-only.

support limit parameter in geocoder #1258

Open
wants to merge 3 commits into
base: publisher-production
Choose a base branch
from

Conversation

tomap
Copy link

@tomap tomap commented May 10, 2018

geocoder api supports between 1 and 10 search results. Default value is 5.
Now, it's possible to set the limit

@davidtheclark davidtheclark changed the base branch from mb-pages to publisher-production January 16, 2019 21:57
@tomap
Copy link
Author

tomap commented May 30, 2019

Hey, you published a new version recently, could you look at the PR?

@tomap tomap requested a review from a team February 19, 2021 15:26
@riastrad
Copy link
Contributor

riastrad commented Mar 3, 2021

Thanks for flagging this, @tomap. As far as I can tell this looks like a pretty straightforward change, but before we approve/merge would you mind doing the following two things:

@tomap
Copy link
Author

tomap commented Jan 14, 2022

4 years later, would you mind doing it? I did half the job. I would be very glad if you did the rest of the job :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants