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

Should the Core Location dependency be removed? #4

Closed
irace opened this issue Dec 9, 2014 · 3 comments
Closed

Should the Core Location dependency be removed? #4

irace opened this issue Dec 9, 2014 · 3 comments
Labels

Comments

@irace
Copy link
Contributor

irace commented Dec 9, 2014

This library currently has a Core Location dependency as a “location” parameter seems commonly useful enough to warrant inclusion. I can understand developers not wanting to pull this in so am interested in suggestions on how to best structure this. Perhaps a XExtensionItem+Location subspec that augments the core library with these capabilities?

@irace irace added the question label Dec 9, 2014
@mbbischoff
Copy link
Contributor

That’d be fine but I’m not sure anyone would really mind linking CoreLocation. It’s not like it changes your bundle size or anything.

@irace
Copy link
Contributor Author

irace commented Dec 20, 2014

Considering removing location support altogether for now, considering it's likely not something  would encourage, even though I think there are valid use cases that aren't just sending the user's current location from an app that has permissions to an extension that doesn’t (e.g. sharing a Foursquare check-in to an extension like Day One's).

@mbbischoff
Copy link
Contributor

Considering it's likely not something  would encourage

Yeah, thinking about it a bit, that’s probably true. Like you said, you could be sharing the location of a historic site, or a checkin, and not just where you are, but probably so few people would use that, that it isn’t worth being in the 1.0 anyway.

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

No branches or pull requests

2 participants