Skip to content

Conversation

@wivaku
Copy link
Contributor

@wivaku wivaku commented Jun 29, 2020

Resolves #1063

The reverse query was using Search (requires Type).
This PR adds support for Nearby, which can be used with:
rankby:prominence + radius (=default Nearby combination)
rankby:distance + type/keyword/name (distance + type is very similar to the original Search)

Note: Search returns formatted_address, Nearby returns vicinity.

@wivaku wivaku marked this pull request as ready for review June 29, 2020 12:55
@wivaku
Copy link
Contributor Author

wivaku commented Jul 27, 2020

@jbelien anything I need to do to resolve the reported branch conflict?

I did update the CHANGELOG.md. Originally I had picked v1.1.0 as the new version, but in parallel this was used for the PHP < 7.2 update as well. So I changed CHANGELOG.md and am using v1.2.0 now.

@jbelien
Copy link
Member

jbelien commented Jul 27, 2020

@jbelien anything I need to do to resolve the reported branch conflict?

You shouldn't update the CHANGELOG.md file yourself, you need to merge/rebase the `master branch in your branch.

@wivaku
Copy link
Contributor Author

wivaku commented Jul 27, 2020

I believe that's done, or am I doing that incorrectly (remember: apologies, this is my first pull request...)?

~/code/pull_requests/Geocoder/src/Provider/GoogleMapsPlaces - issue-1063
╰─ git merge master
Already up to date.

@jbelien
Copy link
Member

jbelien commented Jul 27, 2020

You need to pull/download the changes made in the remote/origin repository (from GitHub).
Could you try the following ?

git checkout master
git pull origin master
git checkout issue-1063
git merge master

@wivaku
Copy link
Contributor Author

wivaku commented Jul 27, 2020

After git checkout master and git pull origin master CHANGELOG.md contains 1.0.1 and not 1.1.0 ( < PHP 7.2),
so it seems it is not pulling the latest master. The listed commit is d6f45a4, which is the June 27 commit, so misses the 7 July 4 commits.

╰─ git checkout master

Switched to branch 'master'
Your branch is up to date with 'origin/master'.

╰─ git pull origin master

From github.com:wivaku/Geocoder
 * branch            master     -> FETCH_HEAD
Already up to date.

╰─ git rev-parse --short HEAD
d6f45a4

╰─ git checkout issue-1063

Switched to branch 'issue-1063'
Your branch is up to date with 'origin/issue-1063'.

╰─ git merge master

Already up to date.

@jbelien
Copy link
Member

jbelien commented Jul 27, 2020

Oh sorry! I forgot you were in a fork of our repository. The origin is indeed your fork and not our repository.

Here are the commands:

git remote add upstream git://github.com/geocoder-php/Geocoder.git
git fetch upstream
git checkout master
git pull upstream master
git checkout issue-1063
git merge master

@wivaku
Copy link
Contributor Author

wivaku commented Jul 27, 2020

that was it. Thanks for the very fast & useful responses!

@jbelien jbelien merged commit 0380943 into geocoder-php:master Oct 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Google Maps Places reverse lookup: without type, keyword, name

2 participants