-
Notifications
You must be signed in to change notification settings - Fork 32
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
Fix for compilation error with flutter version 3.29 #84
Conversation
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…r, but FlutterJNI was detached from native C++. Could not send. Channel: nearby_connections. Response ID: 4 Refer from. https://medium.com/@gauravpaudel2013/tried-to-send-a-platform-message-to-flutter-but-flutterjni-was-detached-from-native-c-well-well-c7d9ccf0df96
…ConnectionsPlugin - Removed the legacy plugin registration method `registerWith` as it is no longer needed. - Removed unused `pluginRegistrar` variable. - Eliminated commented-out code related to method invocation within event callbacks for better readability. - Replaced the `//channel.invokeMethod` with `//` for readability in `onConnectionInitiated`, `onConnectionResult`, `onDisconnected`, `onPayloadReceived`, `onPayloadTransferUpdate`, `onEndpointFound`, and `onEndpointLost` callbacks.
lib/src/nearby.dart
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why it is coming as a changed file.
I am unable to find a way to remove the file from Pull request, but its not required to fix compilation error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can try running git pull upstream release --no-rebase
then resolve the merge conflicts (use keep theirs)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
# Conflicts: # android/src/main/java/com/pkmnapps/nearby_connections/NearbyConnectionsPlugin.java # lib/src/nearby.dart
u may wanna run |
This commit refactors the Dart code within the `Nearby` class to improve readability by aligning the code within the `receiveBroadcastStream` listener. Specifically, it ensures consistency by aligning the `case` statements and their corresponding code blocks.
Thanks @rahulmaindargi 🚀 |
#83