Merge pull request #227 from TaskarCenterAtUW/fix-submission-apierror-handling

pass apierror instead of bool

Author: iAKM

GoInfoGame/GoInfoGame/Network/ApiManager.swift

@@ -196,7 +196,7 @@ class ApiManager {
                             }
 
                         default:
-                            completion(.failure(APIError.custom("Unexpected HTTP status code: \(response.statusCode)")))
+                            completion(.failure(APIError(statusCode: response.statusCode)))
                         }
                     } else {
                         completion(.failure(APIError.custom("No valid HTTP response received")))

GoInfoGame/GoInfoGame/data/DatasyncManager.swift

@@ -18,27 +18,27 @@ class DatasyncManager {
     private var isSynching: Bool = false
 
     private let dbInstance = DatabaseConnector.shared
-        
-    func syncDataToOSM(completionHandler: @escaping (Bool) -> Void) {
+    
+    func syncDataToOSM(completionHandler: @escaping (Result<Bool, APIError>)  -> Void) {
         Task {
             do {
               let isSynced = try await syncData()
                 print("Sync finished")
                 if isSynced {
                     print("Sync successful")
                     DispatchQueue.main.async {
-                        completionHandler(true) // Success
+                        completionHandler(.success(true)) // Success
                     }
                 } else {
                     print("Sync failed")
                     DispatchQueue.main.async {
-                        completionHandler(false) // Failure
+                        completionHandler(.failure(APIError.custom("Sync failed. Please try again."))) // Failure
                     }
                 }
             } catch {
                 print("Sync failed: \(error)")
                 DispatchQueue.main.async {
-                    completionHandler(false) // Failure
+                    completionHandler(.failure(error as! APIError)) // Failure
                 }
             }
         }