-
Notifications
You must be signed in to change notification settings - Fork 55
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
Try launching in external app first #43
base: master
Are you sure you want to change the base?
Conversation
Thank you, hopefully this gets a look from the owner soon. |
@pikaju , can this be merged? |
@@ -56,20 +56,33 @@ class MapsLauncher { | |||
return uri; | |||
} | |||
|
|||
static Future<LaunchMode> _launchMode() async => | |||
await supportsLaunchMode(LaunchMode.externalNonBrowserApplication) |
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.
@lukehutch Please help me check if the method supportsLaunchMode
exists. I received this error when trying to build
Error (Xcode): ../../../../.pub-cache/git/flutter-maps-launcher-470dd6965ea83de929851c48723f5745dbafbfcd/lib/maps_launcher.dart:60:13: Error: Method not found: 'supportsLaunchMode'.
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.
Yes, this method is in url_launcher
, which is a dependency of flutter_maps_launcher
.
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.
I did update the deps versions though, as I said in the description, so make sure you update those, then dart pub upgrade
Follows the advice here to try to fix #42.
(The changes are all in maps_launcher.dart, but I also updated the deps, hence the large changeset)