Skip to content

Ios retrocompat#6

Open
flutter-painter wants to merge 5 commits intoAlystrasz:masterfrom
flutter-painter:ios-retrocompat
Open

Ios retrocompat#6
flutter-painter wants to merge 5 commits intoAlystrasz:masterfrom
flutter-painter:ios-retrocompat

Conversation

@flutter-painter
Copy link
Copy Markdown

@Alystrasz first thank you for forking this initially

I presume there are not so many coconuts out there caring for retrocompat,
If by any chance you wish to walk along this dusty road
Indulge me by accepting this PR

Copy link
Copy Markdown
Owner

@Alystrasz Alystrasz left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, I'll have some time next week to review it properly.

public class SwiftBluetoothEnablePlugin: NSObject, FlutterPlugin, CBCentralManagerDelegate {
var centralManager: CBCentralManager!
var lastKnownState: CBManagerState!
var lastKnownState = ""
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What's the difference between not initializing lastKnownState and set it to ""?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

'CBManagerState' is only available in iOS 10.0 or newer
So i avoid using it

@flutter-painter
Copy link
Copy Markdown
Author

Hey @Alystrasz it should be good now,
Please have a look
ps : I know flutter new versions will not support old ios, but some of our users are still using these good all ipad minis, would like to keep them in the loop for a little while

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