Skip to content
This repository was archived by the owner on Jan 28, 2026. It is now read-only.

Add Google component filters for administrative areas, postal codes and countries#60

Merged
chadly merged 6 commits intochadly:masterfrom
daveajrussell:master
Sep 27, 2016
Merged

Add Google component filters for administrative areas, postal codes and countries#60
chadly merged 6 commits intochadly:masterfrom
daveajrussell:master

Conversation

@daveajrussell
Copy link
Contributor

Added some functionality and tests for Google Component Filters.

I've added this because we have a scenario here where we only want to geocode within the UK.

I didn't bother with Route, given that this allows you to geocode without an address, which is required in this implementation.

I also haven't implemented Locality, as I couldn't find a use-case and didn't want to commit untested code.

@daveajrussell
Copy link
Contributor Author

daveajrussell commented Sep 23, 2016

Ah - I am using a C# 6 feature, string interpolation. Let me fix.

@daveajrussell daveajrussell reopened this Sep 23, 2016
@chadly
Copy link
Owner

chadly commented Sep 23, 2016

Thanks for this! I just updated the build server by the way to not freak out on C# 6 features.

Before I merge, can you fix the formatting of the files? It is hard to read the diff since it looks like all the tabs have been converted to spaces.

@daveajrussell
Copy link
Contributor Author

Weird, I hadn't altered the formatting at all... My copy of VS must be doing something funky. I actually had to grab your raw files from source, open them in np++ and paste in my changes to fix the formatting. Hopefully all looks good now.

Convert formatting to tabs instead of spaces
@chadly chadly merged commit 4db1b45 into chadly:master Sep 27, 2016
@chadly
Copy link
Owner

chadly commented Sep 27, 2016

Awesome! Thanks again!

adamhathcock added a commit to adamhathcock/Geocoding.net that referenced this pull request Dec 13, 2016
* master:
  Don't include test project in ilmerged output
  Register google as default geocoder for example app
  Update .gitignore for VS 2015
  Updating readme mapquest issues
  Upgrade Geocoding.Net in Example.sln. Can't reproduce chadly#45 and chadly#48. They are already fixed by commit ecc8cf6
  Split tests for special characters and Street intersections (by ampersand)
  Test MapQuest Commercial API
  Implemented Google Geocoding channel feature (chadly#63)
  Bump version
  Add Google component filters for administrative areas, postal codes and countries (chadly#60)
  Adding Mixed SideOfStreet to MapQuest
  Bump minor version ahead of release
  Fixing space formatting
  Minor change
  Added place_id to GoogleAddress (chadly#52)
  update example output to match current return (chadly#54)

# Conflicts:
#	Google/Google.csproj
#	Tests/MapQuestBatchGeocoderTest.cs
#	geocoding.nuspec
#	src/Geocoding.Google/GoogleGeocoder.cs
#	teamcity.targets
#	test/Geocoding.Tests/GoogleGeocoderTest.cs
#	test/Geocoding.Tests/MapQuestGeocoderTest.cs
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.

2 participants