Feature 5 & 6

OpenWeather/App/Home/HomeScreen.swift

@@ -21,11 +21,7 @@ struct HomeScreen: View {
     private let location: CLLocation = Mock.londonLocation
 
     @ObservedObject var settings: AppSettings = AppSettings.shared
-    @ObservedObject var viewModel: HomeViewModel
-
-    init(viewModel: HomeViewModel) {
-        self.viewModel = viewModel
-    }
+    @StateObject var viewModel: HomeViewModel
 
     var body: some View {
         viewModel.view { content in

OpenWeather/App/OpenWeatherApp.swift

@@ -18,7 +18,7 @@ import SwiftUI
 
 @main
 struct OBOpenWeatherApp: App {
-    @ObservedObject var navigation: Navigation = Navigation.shared
+    @StateObject var navigation: Navigation = Navigation.shared
 
     var body: some Scene {
         WindowGroup {

OpenWeather/App/RootView.swift

@@ -30,11 +30,10 @@ struct RootView: View {
     var body: some View {
         TabView(selection: $navigation.tab) {
             OpenBytesNavigationView(path: navigation.home) {
-                // TODO: Update to Production
                 HomeScreen(
                     viewModel: HomeViewModel(
                         capabilities: .init(
-                            locationProviding: MockLocationProvider(),
+                            locationProviding: LocationProvider(),
                             weatherProviding: OpenWeatherMapWeatherProvider()
                         ),
                         input: .init()
@@ -48,8 +47,14 @@ struct RootView: View {
             }
 
             OpenBytesNavigationView(path: navigation.search) {
-                // TODO: Update to Production
-                SearchLocationScreen(viewModel: .mock)
+                SearchLocationScreen(
+                    viewModel: .init(
+                        capabilities: .init(
+                            locationProviding: LocationProvider()
+                        ),
+                        input: .init()
+                    )
+                )
             }
             .tag(Tab.search)
             .tabItem {
@@ -58,8 +63,15 @@ struct RootView: View {
             }
 
             OpenBytesNavigationView(path: navigation.summary) {
-                // TODO: Update to Production
-                SummaryScreen(viewModel: .mock)
+                SummaryScreen(
+                    viewModel: .init(
+                        capabilities: .init(
+                            locationProviding: LocationProvider(),
+                            weatherProviding: OpenWeatherMapWeatherProvider()
+                        ),
+                        input: .init()
+                    )
+                )
             }
             .tag(Tab.summary)
             .tabItem {

OpenWeather/App/RootViewModel.swift

@@ -0,0 +1,8 @@
+//
+//  OpenWeather template generated by OpenBytes on 30/04/2023.
+//
+// Created by Ahmed Shendy.
+//  RootViewModel.swift
+//
+
+import Foundation

OpenWeather/App/Search/SearchLocationScreen.swift

@@ -18,11 +18,7 @@ import OpenBytesNavigation
 import SwiftUI
 
 struct SearchLocationScreen: View {
-    @ObservedObject var viewModel: SearchLocationViewModel
-
-    init(viewModel: SearchLocationViewModel) {
-        self.viewModel = viewModel
-    }
+    @StateObject var viewModel: SearchLocationViewModel
 
     var body: some View {
         viewModel.view { content in

OpenWeather/App/Search/SearchLocationViewModel.swift

@@ -86,23 +86,28 @@ final class SearchLocationViewModel: ViewModel<
             }
     }
 
-    private func getLocations(searchText: String) {
-        searchTask?.cancel()
+private func getLocations(searchText: String) {
+    searchTask?.cancel()
 
-        searchTask = Task {
-            do {
-                guard searchTask?.isCancelled == false else { return }
+    guard searchText.isEmpty == false else {
+        self.result = []
+        return
+    }
+
+    searchTask = Task {
+        do {
+            guard searchTask?.isCancelled == false else { return }
 
-                let result = try await capabilities.getLocations(query: searchText)
+            let result = try await capabilities.getLocations(query: searchText)
 
-                guard searchTask?.isCancelled == false else { return }
+            guard searchTask?.isCancelled == false else { return }
 
-                await MainActor.run {
-                    self.result = result
-                }
-            } catch {
-                errorHandler.handle(error: error)
+            await MainActor.run {
+                self.result = result
             }
+        } catch {
+            errorHandler.handle(error: error)
         }
     }
 }
+}

OpenWeather/App/Summary/SummaryScreen.swift

@@ -9,13 +9,9 @@ import SwiftUI
 import CoreLocation
 
 struct SummaryScreen: View {
-    private let locations: [CLLocation] = [.london, .cairo, .newYork]
+    @StateObject var viewModel: SummaryViewModel
 
-    @ObservedObject var viewModel: SummaryViewModel
-
-    init(viewModel: SummaryViewModel) {
-        self.viewModel = viewModel
-    }
+    private let locations: [CLLocation] = [.london]
 
     var body: some View {
         viewModel.view { content in

OpenWeather/App/Summary/SummaryViewModel.swift

@@ -17,17 +17,17 @@ final class SummaryViewModel: ViewModel<
     struct Capabilities {
         static var mock: Capabilities {
             .init(
-                weatherProviding: MockWeatherProvider(),
-                locationProviding: MockLocationProvider()
+                locationProviding: MockLocationProvider(),
+                weatherProviding: MockWeatherProvider()
             )
         }
 
         private var weatherProviding: WeatherProviding
         private var locationProviding: LocationProviding
 
-        init(weatherProviding: WeatherProviding, locationProviding: LocationProviding) {
-            self.weatherProviding = weatherProviding
+        init(locationProviding: LocationProviding, weatherProviding: WeatherProviding) {
             self.locationProviding = locationProviding
+            self.weatherProviding = weatherProviding
         }
 
         func locationName(for location: CLLocation) async throws -> String {

OpenWeather/Data/Adapters/LocationAdapter.swift

@@ -9,7 +9,7 @@ import Foundation
 import CoreLocation
 
 enum LocationAdapter: Adaptable {
-    static func device(from: NetworkLocation) -> DeviceLocation {
+    static func device(from: LocationData) -> DeviceLocation {
         return DeviceLocation(
             name: from.name,
             location: from.location

OpenWeather/Data/Device/DeviceLocation.swift

@@ -12,4 +12,7 @@ struct DeviceLocation: Identifiable {
     let id = UUID()
     let name: String
     let location: CLLocation
+
+    var lat: Double { location.coordinate.latitude }
+    var long: Double { location.coordinate.longitude }
 }

OpenWeather/Data/Network/LocationData.swift

@@ -0,0 +1,43 @@
+//
+//  OpenWeather template generated by OpenBytes on 22/03/2023.
+//
+// Created by Ahmed Shendy.
+//  LocationData.swift
+//
+
+import Foundation
+import CoreLocation
+import CryptoKit
+
+struct LocationData {
+    let name: String
+    let location: CLLocation
+
+    var lat: Double { location.coordinate.latitude }
+    var long: Double { location.coordinate.longitude }
+
+    init(name: String, location: CLLocation) {
+        self.name = name
+        self.location = location
+    }
+}
+
+extension LocationData: CustomStringConvertible {
+    var description: String {
+        """
+{
+    name: \(name),
+    latitude: \(lat),
+    longitude: \(long)
+}
+"""
+    }
+}
+
+extension LocationData: Equatable {
+    static func == (lhs: Self, rhs: Self) -> Bool {
+        lhs.name == rhs.name
+        && lhs.lat == rhs.lat
+        && lhs.long == rhs.long
+    }
+}

OpenWeather/Info.plist

@@ -18,6 +18,10 @@
 	<string>1.0</string>
 	<key>CFBundleVersion</key>
 	<string>1</string>
+	<key>NSLocationAlwaysAndWhenInUseUsageDescription</key>
+	<string>This app requires access to your location at all times to provide you with accurate weather forecasts and alerts for your area.</string>
+	<key>NSLocationWhenInUseUsageDescription</key>
+	<string>This app requires access to your location while you are using it to provide you with accurate weather forecasts and alerts for your area.</string>
 	<key>UILaunchScreen</key>
 	<array>
 		<dict>

OpenWeather/Networking/Implementation/Location/LocationProvider.swift

@@ -18,41 +18,14 @@ struct LocationProvider: LocationProviding {
     }
 
     private func locationName(for location: CLLocation) async throws -> LocationNameResponse {
-        return try await withCheckedThrowingContinuation { continuation in
-            locationName(
-                for: location,
-                completion: { placemark, error in
-                    if let error = error {
-                        continuation.resume(
-                            throwing: LocationProvidingError.failure(reason: error.localizedDescription)
-                        )
-                    } else {
-                        if let locationName = placemark?.name {
-                            continuation.resume(
-                                returning: LocationNameResponse(name: locationName)
-                            )
-                        } else {
-                            continuation.resume(
-                                throwing: LocationProvidingError.noSuchPlace
-                            )
-                        }
-                    }
-                }
-            )
-        }
-    }
+        let placemarks = try await geocoder.reverseGeocodeLocation(location)
 
-    private func locationName(
-        for location: CLLocation,
-        completion: @escaping (CLPlacemark?, Error?) -> Void
-    ) {
-        geocoder.reverseGeocodeLocation(location) { placemarks, error in
-            if let error = error as? NSError {
-                completion(nil, error)
-            } else {
-                completion(placemarks?.first, nil)
-            }
+        guard let name = placemarks.first?.name
+        else {
+            throw LocationProvidingError.noSuchPlace
         }
+
+        return LocationNameResponse(name: name)
     }
 
     func locations(for address: String) async throws -> [DeviceLocation] {
@@ -71,14 +44,14 @@ struct LocationProvider: LocationProviding {
                             throwing: LocationProvidingError.failure(reason: error.localizedDescription)
                         )
                     } else {
-                        let result: [NetworkLocation] = placemarks
-                            .compactMap { placemark -> NetworkLocation? in
+                        let result: [LocationData] = placemarks
+                            .compactMap { placemark -> LocationData? in
                                 guard
                                     let name = placemark.name,
                                     let location = placemark.location
                                 else { return nil }
 
-                                return NetworkLocation(
+                                return LocationData(
                                     name: name,
                                     location: location
                                 )

OpenWeather/Networking/Interface/Location/LocationProviding+Models.swift

@@ -13,5 +13,5 @@ struct LocationNameResponse {
 }
 
 struct LocationSearchResponse {
-    let result: [NetworkLocation]
+    let result: [LocationData]
 }