Skip to content

Conversation

@n-sviridenko
Copy link

Hello!

I'm understanding that you declined this feature. But despite the fact that this parameter isn't common, it's very useful for the localization, because we don't have a good php library for address localization yet. I'm sure that most of people are using google maps provider and also they're using this parameter. And it will be better to use your solution instead of writing own just for one property.

If you are not agree, would you please to add a raw response data of the providers?

Thank you for avance!

@n-sviridenko n-sviridenko force-pushed the feature/formatted-address branch from a3226cb to f37ab76 Compare August 19, 2016 18:11
@n-sviridenko n-sviridenko force-pushed the feature/formatted-address branch from f37ab76 to a6df761 Compare August 19, 2016 20:37
@n-sviridenko
Copy link
Author

Also, you can create a compatibility table for each provider and each ability. It will warn users.

Copy link
Member

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Im not sure this is the correct approach. Though I think we should have this feature in v4.0.0. I've created an issue for this. See #558

I'll keep this PR on hold.

@Nyholm Nyholm added the on hold label Dec 22, 2016
@Nyholm Nyholm added this to the 4.0.0 milestone Dec 22, 2016
@Nyholm Nyholm removed the on hold label May 26, 2017
@Nyholm
Copy link
Member

Nyholm commented May 26, 2017

Hey @n-sviridenko

There has been a lots of changes in master since you made this PR. There is now a custom Google response where this change should be added. Can you rebase this PR and make sure you add the changes to the GoogleAddress instead of the shared one?

This PR is no longer on hold.

@Nyholm
Copy link
Member

Nyholm commented May 30, 2017

Closed by #655

@Nyholm Nyholm closed this May 30, 2017
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.

2 participants