UIAlertView is deprecated about armchair HOT 11 CLOSED

I will entertain a pull request that maintains backwards compatibility.

from armchair.

@jlaws just set usesAlertController to true for the moment

@coneybeare Maybe we could change the default value of usesAlertController to true? or (according to iOS version > 9)

from armchair.

I like that idea of setting the default to true based on operatingSystemVersion

from armchair.

Finally got around to looking at this thread again. My concern was the build warning:

Armchair-Swift.h:130:20: 'UIAlertView' is deprecated: first deprecated in iOS 9.0 - UIAlertView is deprecated. Use UIAlertController with a preferredStyle of UIAlertControllerStyleAlert instead

Considering that 95% or 97.5% of devices are running iOS 8+ it is probably safe to make the min target for the library iOS 8 and use alert controllers exclusively. Just create a fork and leave the old cocoapods for people who want to target earlier versions.

Another option is to suppress the warning inside the library.

from armchair.

I agree with @jlaws solution. If you really don't want to target only iOS8+, you should suppress that warning directly in the library.

from armchair.

I'd consider a pull request of that warning suppression

from armchair.

Please fix that warning. Also iOS8+ only is fine.

from armchair.

This really needs to be fixed, every time I do a clean build it gives me the warning. At this point, its pretty annoying. I moved to this library so that it would be modern and a Swift solution to Appirater.

from armchair.

@KMessina1 I will entertain any pull request surpassing the warning, please feel free to contribute.

from armchair.

@coneybeare @jlaws any update on this? It's still the same.

from armchair.

UIAlertView has already been completely removed from source in master.

from armchair.