dpa99c / phonegap-launch-navigator

Phonegap/Cordova plugin which launches native route navigation apps for Android, iOS and Windows
369 stars 131 forks source link

Integration with Cabify not working as expected #226

Open giosgu opened 4 years ago

giosgu commented 4 years ago

I'm submitting a ... (check one with "x"):

Bug report

Current behavior: Hi there, first of all, thanks for your great work. It seems that integration with Cabify is not working properly. If you choose that app from the app list, it launches Cafiby's initial page instead of showing the route to destination:

image

Here is an ionic 3 sample project to reproduce this behaviour: https://github.com/giosgu/launchNavigatorCabifyIssue

It is working properly with Google Maps, Uber, Waze, but not Cabify. It happens both in Android and iOS.

I'd appreciate if you could take a look at this, many thanks!

Expected behavior: When user selects Cabify from the app list, it should show the route and allow you to select the car: image

Environment information

$ ionic info

Ionic:

ionic (Ionic CLI) : 4.12.0 (C:\AppData\Roaming\nvm\v10.16.0\node_modules\ionic) Ionic Framework : ionic-angular 3.9.5 @ionic/app-scripts : 3.2.2

Cordova:

cordova (Cordova CLI) : 8.1.2 (cordova-lib@8.1.1) Cordova Platforms : not available Cordova Plugins : not available

System:

Android SDK Tools : 26.1.1 (C:\Users\MT30886835\AppData\Local\Android\Sdk) NodeJS : v10.16.0 (C:\Program Files\nodejs\node.exe) npm : 6.9.0 OS : Windows 10

Related code:

  comoLlegar(){
    this.launchNavigator.navigate("-34.545073, -58.449725", {
      appSelection: {
        dialogHeaderText: "Seleccione la app asociada",
        cancelButtonText: "Cancelar",
      }
    })
    .then(
      success => console.log('Launched navigator'),
      error => alert('Hubo un error al comenzar la navegación ' +  error)
    );

  }

// Paste any relevant JS/native console output here



Feature request

Documentation issue

dpa99c commented 4 years ago

It's possible that the URL scheme of Cabify has been changed which would break the integration with this plugin. This has happened before (see #155). I need to investigate and compare the current URL scheme being used by this plugin with the currently documented Cabify URL scheme for mobile deep linking. Can you provide the version numbers of the Cabify app on both Android & iOS in which this issue was observed?

giosgu commented 4 years ago

The issue is observed in the latest available version of Cabify, 7.11.0 (Android & iOS)