fix location fetching issues (#66)

patrickdemers6 · Dec 1, 2022 · f21b9a9 · f21b9a9
1 parent f294b45
commit f21b9a9
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 9 deletions.
diff --git a/App.js b/App.js
@@ -8,9 +8,15 @@ import Main from './src/Main';
 
 const App = () => {
   React.useEffect(() => {
-    (async () => {
-      await Location.requestForegroundPermissionsAsync();
-    })();
+    const getLocationPermission = async () => {
+      try {
+        await Location.requestForegroundPermissionsAsync();
+      } catch (e) {
+        console.log('requestForegroundPermissionsAsync failed');
+      }
+    };
+
+    setTimeout(getLocationPermission, 1000);
   }, []);
 
   return (

diff --git a/src/components/Map/Map.js b/src/components/Map/Map.js
@@ -47,11 +47,13 @@ const Map = () => {
 
   const getCurrentUserLocation = () => {
     (async () => {
-      const location = await Location.getCurrentPositionAsync();
-      dispatcher?.setUserLocation(location);
-    })().catch(() => {
-      console.log('No location permissions granted.');
-    });
+      try {
+        const location = await Location.getCurrentPositionAsync();
+        dispatcher?.setUserLocation(location);
+      } catch (e) {
+        console.log('No location permissions granted.');
+      }
+    })();
   };
 
   const moveMapToUser = () => {
@@ -70,9 +72,12 @@ const Map = () => {
   };
 
   React.useEffect(updateVehiclesOnForeground, [route]);
-  React.useEffect(getCurrentUserLocation, []);
   React.useEffect(moveMapToUser, [location]);
 
+  // get current user location on load and if dispatcher changes
+  React.useEffect(getCurrentUserLocation, []);
+  React.useEffect(getCurrentUserLocation, [dispatcher]);
+
   return (
     <View style={{ width, height }}>
       <MapView