-
Notifications
You must be signed in to change notification settings - Fork 83
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
iOS13 Compatibility #89
Comments
Any thoughts on this @acmacalister ? Its just generally much better to have no warnings in the code so that developers are able to spot new warnings during coding instantly without having to read through a list of existing warnings. Thanks. |
Thanks for fixing this. Any idea when it will be pulled into master? Thanks. |
@ddaddy any news for when this will be hitting master? |
Sorry, I'm not the owner of this. I submitted the pull request but @acmacalister needs to pull it. |
Ok cool. Will wait for @acmacalister to merge into master then. |
There are 11 open pull requests open since 2015, and no commit on this repo for over 2 years. I'm guessing it's a dead repo. Interestingly, Sonos just forked it: https://github.com/sonos/jetfire |
There are a few new warnings as a result of iOS13.
The main culprit seems to be...
'SecTrustEvaluate' is deprecated: first deprecated in iOS 13.0
Screenshots of the code with these warnings...
The text was updated successfully, but these errors were encountered: