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

Nice to have: NetworkObserver #61

Open
agiguere opened this issue May 30, 2016 · 3 comments
Open

Nice to have: NetworkObserver #61

agiguere opened this issue May 30, 2016 · 3 comments

Comments

@agiguere
Copy link

Originally the Apple WWDC 2015 sample code was provided with a NetworkObserver class, pretty useful

I don't know why you drop it or maybe you forget to add it ?

alex

@mcmurrym
Copy link
Contributor

mcmurrym commented Jul 6, 2016

the NetworkObserver only works on iOS, the library works on all platforms and extension targets. We may consider re-adding it but it would not be a part of the framework.

@agiguere
Copy link
Author

ah I see, yeah that could be bundle as an add-on for iOS only

thanks

@ffried
Copy link
Contributor

ffried commented Jan 24, 2017

Why don't you include it enclosed with #if os(iOS) [...] #endif? That way it would only be available on iOS.

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

No branches or pull requests

3 participants