YoussefHenna / expo-mapbox-navigation

MIT License
3 stars 0 forks source link

Crash on IOS and sometime on Android Too #5

Open jacquesngomeheffa opened 1 month ago

jacquesngomeheffa commented 1 month ago

Dear Youssef,

I really appreciate the effort of your hard work. I have been testing your last release on both IOS & Android. I have encountered some issue on both.

@rnmapbox/maps & expo-mapbox-navigation configuration in app.json

 [
        "@rnmapbox/maps",
        {
          "RNMapboxMapsVersion": "11.3.0",
          "RNMapboxMapsImpl": "mapbox",
          "RNMapboxMapsDownloadToken": "sk.xxxxxx"
        }
      ],

[
        "@youssefhenna/expo-mapbox-navigation",
        {
          "accessToken": "pk.xxxxxxxx"
        }
      ],

App initialisation

<MapboxNavigationView
          coordinates={coordinates}
          style={styles.container}
          locale={"fr"}
          onRouteProgressChanged={(event) => {}}
          onWaypointArrival={(event) => { }}
          onCancelNavigation={(event) => {
            navigation.goBack();
          }}
          onFinalDestinationArrival={async () => {}}
/>

IOS crash (Can be simulated each time) steps below:

  1. Open the navigation view
  2. Close it
  3. re-open it (Crash) I reloaded the app again , repeat the steps above => app crashes again I don't know if I'm doing something wrong?

Sometime the navigationview does not load (pic below) image

Android crash

sometime, when Android crashed i received the error below (see pic) image

Android bug

Sometime the navigation icon does not appear on the top of the navigation path (see pic) image

Navigation notification When I open then navigationview, it triggers a notification to appear, but when i close the navigationview, it does not disappear until I close it by my self? I don't know if it is a bug, but i wanted to let you know about it (see pic ) image

Android & IOS It looks like the navigation icon is different on both Platform?

Thank you and great job!

jacquesngomeheffa commented 1 day ago

Dear @YoussefHenna Any update for this issue please?

YoussefHenna commented 1 day ago

Hi,

I did address some of these issue, but not all, that's why I'm keeping the ticket open for now.

If you can test the latest version and let me know which issues are still reproducible, that would be very helpful. Thanks!

jacquesngomeheffa commented 1 day ago

Hi,

I did address some of these issue, but not all, that's why I'm keeping the ticket open for now.

If you can test the latest version and let me know which issues are still reproducible, that would be very helpful. Thanks!

Thanks, I will let you know