Ryan you are an idiot! Fixed for real now :D

Author: stillie

.github/workflows/build-test-and-release.yml

@@ -31,6 +31,7 @@ jobs:
           dart run build_runner build -d
       - run: flutter test
       - run: flutter build apk --release
+      - run: flutter build apk --debug
       - name: Push to Releases
         uses: ncipollo/release-action@v1
         with:

lib/home/current_weather_repository.dart

@@ -15,11 +15,11 @@ class WeatherRepository extends ChangeNotifier {
 
   void getWeatherForCurrentLocation() async {
     final locationData = await locationRepo.initLocationData();
-    if (locationData == null) return;
+    print("Calling current whether ${locationData?.latitude} ${locationData?.longitude} ");
     try {
       currentWeather = await _apiClient.fetchCurrentWeather(
-        locationData.latitude ?? 0,
-        locationData.longitude ?? 0,
+        locationData?.latitude ?? 0,
+        locationData?.longitude ?? 0,
         useDummyData: true,
       );
     } catch (error) {

lib/home/forecast_weather_repository.dart

@@ -19,10 +19,10 @@ class ForecastWeatherRepository extends ChangeNotifier {
   void getWeatherForcast() async {
     try {
       final locationData = await locationRepo.initLocationData();
-      if (locationData == null) return;
+      
       forcastWeather = await _apiClient.fetchForcastWeather(
-        locationData.latitude ?? 0.0,
-        locationData.longitude ?? 0.0,
+        locationData?.latitude ?? 0.0,
+        locationData?.longitude ?? 0.0,
         useDummyData: true,
       );
       if (forcastWeather == null) throw WeatherRequestFailure();

lib/location/location.dart

@@ -15,6 +15,7 @@ class LocationRepository extends ChangeNotifier {
     // This is terrible! There is a known bug that crashes the app because the android system for location has not been enabled by the time it gets here
     // Note: This only happened when running the app in release mode (-_-)
     // See: https://stackoverflow.com/questions/67663357/flutter-location-package-generates-platform-exception-on-call-to-serviceenabled
+
     try {
       _serviceEnabled = await locationService.serviceEnabled();
     } on PlatformException catch (err) {
@@ -27,7 +28,6 @@ class LocationRepository extends ChangeNotifier {
     if (!_serviceEnabled) {
       _serviceEnabled = await locationService.requestService();
       if (!_serviceEnabled) {
-        notifyListeners();
         return null;
       }
     }
@@ -36,11 +36,10 @@ class LocationRepository extends ChangeNotifier {
     if (_permissionGranted == PermissionStatus.denied) {
       _permissionGranted = await locationService.requestPermission();
       if (_permissionGranted != PermissionStatus.granted) {
-        notifyListeners();
         return null;
       }
     }
-
+    hasPermissions = true;
     _locationData = await locationService.getLocation();
     notifyListeners();
     return _locationData;

lib/main.dart

@@ -16,10 +16,17 @@ void main() async {
   runApp(
     MultiProvider(
       providers: [
+        ChangeNotifierProvider<LocationRepository>(
+          create: (context) {
+            return locationRepository;
+          },
+        ),
         ChangeNotifierProvider<WeatherRepository>(
           create: (context) {
             WeatherRepository repository = WeatherRepository(
-                apiClient: apiClient, locationRepo: locationRepository);
+              apiClient: apiClient,
+              locationRepo: locationRepository,
+            );
 
             repository.getWeatherForCurrentLocation();
             return repository;
@@ -29,16 +36,10 @@ void main() async {
           create: (context) {
             ForecastWeatherRepository repository = ForecastWeatherRepository(
                 apiClient: apiClient, locationRepo: locationRepository);
-
             repository.getWeatherForcast();
             return repository;
           },
         ),
-        ChangeNotifierProvider<LocationRepository>(
-          create: (context) {
-            return locationRepository;
-          },
-        )
       ],
       child: const WeatherApp(),
     ),