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.

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

Added 'key' param #9

Merged
merged 1 commit into from
Nov 8, 2015
Merged

Added 'key' param #9

merged 1 commit into from
Nov 8, 2015

Conversation

mguentner
Copy link
Contributor

Since 15th September 2015, the Nominatim Service
provided by MapQuest requires an AppKey.

See http://open.mapquestapi.com/nominatim/ for more
information.

Signed-off-by: Maximilian Güntner maximilian.guentner@gmail.com

Since 15th September 2015, the Nominatim Service
provided by MapQuest requires an AppKey.

See http://open.mapquestapi.com/nominatim/ for more
information.

Signed-off-by: Maximilian Güntner <maximilian.guentner@gmail.com>
@Smarre
Copy link

Smarre commented Oct 18, 2015

Is this project going to be maintained at all? I’m going to merge this to my fork, guess I’ll merge other pull requests here to there too...

@mguentner
Copy link
Contributor Author

Please do so. I also use a fork of your repo currently in production:

https://github.com/mguentner/ruby-nominatim

@Smarre
Copy link

Smarre commented Oct 19, 2015

Did. They’re all in master branch so consequently it should show up in my PR... I wonder if I should just publish a gem like ruby-nominatim and call it fork of this project in order to get this mess sorted out.

@mguentner
Copy link
Contributor Author

Sounds good. 👍

@Smarre
Copy link

Smarre commented Oct 22, 2015

Done: https://github.com/Smarre/ruby-nominatim

I’ll add some migration/maintenance commits and look for the release then.

I unforked the repository (i.e. deleted and recreated it) so it does not have dependency to this project any more.

@mguentner
Copy link
Contributor Author

Awesome, will change the repo in my Gemfiles asap 💪

jakubsvehla added a commit that referenced this pull request Nov 8, 2015
@jakubsvehla jakubsvehla merged commit 4134bf8 into jakubsvehla:master Nov 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants