Compare commits
9
Commits
v1.0.37
...
v1.0.44-46
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
1834763f19 | ||
|
|
62c0469669 | ||
|
|
3d6e1624e9 | ||
|
|
b1a34105f1 | ||
|
|
72db88b25c | ||
|
|
97f0eb56d5 | ||
|
|
c6d6392656 | ||
|
|
958fb6eaa6 | ||
|
|
2c806ef14e |
@@ -10,6 +10,7 @@ plugins {
|
||||
|
||||
if (file("google-services.json").exists()) {
|
||||
apply(plugin = "com.google.gms.google-services")
|
||||
apply(plugin = "com.google.firebase.crashlytics")
|
||||
}
|
||||
|
||||
val keystorePropertiesFile = rootProject.file("keystore.properties")
|
||||
@@ -33,8 +34,8 @@ android {
|
||||
applicationId = "pl.firmatpp.kierowca"
|
||||
minSdk = 26
|
||||
targetSdk = 35
|
||||
versionCode = 38
|
||||
versionName = "1.0.37"
|
||||
versionCode = 46
|
||||
versionName = "1.0.44"
|
||||
setProperty("archivesBaseName", "pl.firmatpp.kierowca")
|
||||
testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
|
||||
|
||||
@@ -123,6 +124,7 @@ dependencies {
|
||||
implementation(libs.compose.ui.tooling.preview)
|
||||
implementation(libs.coroutines.android)
|
||||
implementation(platform(libs.firebase.bom))
|
||||
implementation(libs.firebase.crashlytics)
|
||||
implementation(libs.firebase.messaging)
|
||||
implementation(libs.okhttp)
|
||||
implementation(libs.okhttp.logging)
|
||||
|
||||
@@ -10,6 +10,7 @@
|
||||
android:allowBackup="false"
|
||||
android:icon="@mipmap/ic_launcher"
|
||||
android:label="@string/app_name"
|
||||
android:name=".DriverApplication"
|
||||
android:roundIcon="@mipmap/ic_launcher_round"
|
||||
android:supportsRtl="true"
|
||||
android:theme="@style/Theme.TppKierowca">
|
||||
|
||||
@@ -0,0 +1,12 @@
|
||||
package pl.firmatpp.kierowca
|
||||
|
||||
import android.app.Application
|
||||
import pl.firmatpp.kierowca.diagnostics.AppDiagnostics
|
||||
|
||||
class DriverApplication : Application() {
|
||||
override fun onCreate() {
|
||||
super.onCreate()
|
||||
AppDiagnostics.installFirebaseCrashlytics()
|
||||
AppDiagnostics.log("app_started")
|
||||
}
|
||||
}
|
||||
@@ -1,11 +1,12 @@
|
||||
package pl.firmatpp.kierowca
|
||||
|
||||
import android.os.Bundle
|
||||
import androidx.activity.ComponentActivity
|
||||
import androidx.activity.compose.setContent
|
||||
import androidx.compose.runtime.getValue
|
||||
import androidx.compose.runtime.mutableStateOf
|
||||
import androidx.compose.runtime.setValue
|
||||
import androidx.activity.ComponentActivity
|
||||
import androidx.activity.compose.setContent
|
||||
import androidx.lifecycle.compose.collectAsStateWithLifecycle
|
||||
import androidx.lifecycle.viewmodel.compose.viewModel
|
||||
import pl.firmatpp.kierowca.ui.DriverApp
|
||||
import pl.firmatpp.kierowca.ui.DriverViewModel
|
||||
@@ -13,14 +14,22 @@ import pl.firmatpp.kierowca.ui.theme.TppKierowcaTheme
|
||||
|
||||
class MainActivity : ComponentActivity() {
|
||||
private var notificationRouteId by mutableStateOf<String?>(null)
|
||||
private var notificationLeaveRequestId by mutableStateOf<String?>(null)
|
||||
|
||||
override fun onCreate(savedInstanceState: Bundle?) {
|
||||
super.onCreate(savedInstanceState)
|
||||
notificationRouteId = intent.getStringExtra(EXTRA_ROUTE_ID)
|
||||
notificationLeaveRequestId = intent.getStringExtra(EXTRA_LEAVE_REQUEST_ID)
|
||||
setContent {
|
||||
TppKierowcaTheme {
|
||||
val viewModel: DriverViewModel = viewModel()
|
||||
DriverApp(viewModel = viewModel, initialRouteId = notificationRouteId)
|
||||
val state by viewModel.state.collectAsStateWithLifecycle()
|
||||
|
||||
TppKierowcaTheme(themeMode = state.themeMode) {
|
||||
DriverApp(
|
||||
viewModel = viewModel,
|
||||
initialRouteId = notificationRouteId,
|
||||
initialLeaveRequestId = notificationLeaveRequestId,
|
||||
)
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -29,9 +38,11 @@ class MainActivity : ComponentActivity() {
|
||||
super.onNewIntent(intent)
|
||||
setIntent(intent)
|
||||
notificationRouteId = intent.getStringExtra(EXTRA_ROUTE_ID)
|
||||
notificationLeaveRequestId = intent.getStringExtra(EXTRA_LEAVE_REQUEST_ID)
|
||||
}
|
||||
|
||||
companion object {
|
||||
const val EXTRA_ROUTE_ID = "pl.firmatpp.kierowca.EXTRA_ROUTE_ID"
|
||||
const val EXTRA_LEAVE_REQUEST_ID = "pl.firmatpp.kierowca.EXTRA_LEAVE_REQUEST_ID"
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1,6 +1,7 @@
|
||||
package pl.firmatpp.kierowca.data
|
||||
|
||||
import java.io.IOException
|
||||
import java.net.UnknownHostException
|
||||
import retrofit2.HttpException
|
||||
|
||||
enum class ApiErrorKind {
|
||||
@@ -26,6 +27,11 @@ data class ApiError(
|
||||
object ApiErrorMapper {
|
||||
fun map(throwable: Throwable): ApiError =
|
||||
when (throwable) {
|
||||
is UnknownHostException -> ApiError(
|
||||
kind = ApiErrorKind.Network,
|
||||
message = "Brak internetu lub połączenia z serwerem. Sprawdź zasięg i spróbuj ponownie.",
|
||||
retryable = true,
|
||||
)
|
||||
is IOException -> ApiError(
|
||||
kind = ApiErrorKind.Network,
|
||||
message = "Nie udało się połączyć z serwerem. Operacja nie została potwierdzona.",
|
||||
@@ -39,6 +45,9 @@ object ApiErrorMapper {
|
||||
)
|
||||
}
|
||||
|
||||
fun shouldReportNonFatal(throwable: Throwable): Boolean =
|
||||
map(throwable).kind != ApiErrorKind.Network
|
||||
|
||||
fun mapHttpStatus(statusCode: Int, body: String?): ApiError {
|
||||
val code = body?.let { """"code"\s*:\s*"([^"]+)"""".toRegex().find(it)?.groupValues?.getOrNull(1) }
|
||||
val message = body?.let { """"message"\s*:\s*"([^"]+)"""".toRegex().find(it)?.groupValues?.getOrNull(1) }
|
||||
|
||||
@@ -0,0 +1,30 @@
|
||||
package pl.firmatpp.kierowca.data
|
||||
|
||||
import android.content.Context
|
||||
import androidx.datastore.core.DataStore
|
||||
import androidx.datastore.preferences.core.Preferences
|
||||
import androidx.datastore.preferences.core.edit
|
||||
import androidx.datastore.preferences.core.stringPreferencesKey
|
||||
import androidx.datastore.preferences.preferencesDataStore
|
||||
import kotlinx.coroutines.flow.Flow
|
||||
import kotlinx.coroutines.flow.map
|
||||
import pl.firmatpp.kierowca.ui.theme.AppThemeMode
|
||||
|
||||
private val Context.driverAppPreferencesDataStore by preferencesDataStore(name = "driver_app_preferences")
|
||||
|
||||
class AppPreferencesStore(
|
||||
private val dataStore: DataStore<Preferences>,
|
||||
) {
|
||||
constructor(context: Context) : this(context.driverAppPreferencesDataStore)
|
||||
|
||||
private val themeModeKey = stringPreferencesKey("theme_mode")
|
||||
|
||||
val themeMode: Flow<AppThemeMode> = dataStore.data
|
||||
.map { preferences -> AppThemeMode.fromStoredValue(preferences[themeModeKey]) }
|
||||
|
||||
suspend fun setThemeMode(themeMode: AppThemeMode) {
|
||||
dataStore.edit { preferences ->
|
||||
preferences[themeModeKey] = themeMode.storedValue
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -14,8 +14,11 @@ import pl.firmatpp.kierowca.data.api.MobileDriverApi
|
||||
import pl.firmatpp.kierowca.data.model.BootstrapResponse
|
||||
import pl.firmatpp.kierowca.data.model.BroadcastAuthBody
|
||||
import pl.firmatpp.kierowca.data.model.BroadcastAuthResponse
|
||||
import pl.firmatpp.kierowca.data.model.CancelLeaveRequestBody
|
||||
import pl.firmatpp.kierowca.data.model.DispatchSheetUploadResponse
|
||||
import pl.firmatpp.kierowca.data.model.CreateLeaveRequestBody
|
||||
import pl.firmatpp.kierowca.data.model.DriverDto
|
||||
import pl.firmatpp.kierowca.data.model.DriverLeaveRequestDto
|
||||
import pl.firmatpp.kierowca.data.model.NotificationPreferencesBody
|
||||
import pl.firmatpp.kierowca.data.model.NotificationPreferencesDto
|
||||
import pl.firmatpp.kierowca.data.model.OtpResponse
|
||||
@@ -51,6 +54,18 @@ class DriverRepository(
|
||||
suspend fun route(routeId: String): RouteResponse =
|
||||
api.route(authHeader(requireToken()), routeId)
|
||||
|
||||
suspend fun leaveRequests(): List<DriverLeaveRequestDto> =
|
||||
api.leaveRequests(authHeader(requireToken())).data
|
||||
|
||||
suspend fun leaveRequest(id: String): DriverLeaveRequestDto =
|
||||
api.leaveRequest(authHeader(requireToken()), id).data
|
||||
|
||||
suspend fun createLeaveRequest(dateFrom: String, dateTo: String, type: String, note: String?): DriverLeaveRequestDto =
|
||||
api.createLeaveRequest(authHeader(requireToken()), CreateLeaveRequestBody(dateFrom, dateTo, type, note)).data
|
||||
|
||||
suspend fun cancelLeaveRequest(id: String, comment: String? = null): DriverLeaveRequestDto =
|
||||
api.cancelLeaveRequest(authHeader(requireToken()), id, CancelLeaveRequestBody(comment)).data
|
||||
|
||||
suspend fun completeRoute(routeId: String) =
|
||||
api.completeRoute(authHeader(requireToken()), routeId)
|
||||
|
||||
|
||||
@@ -5,8 +5,13 @@ import okhttp3.RequestBody
|
||||
import pl.firmatpp.kierowca.data.model.BootstrapResponse
|
||||
import pl.firmatpp.kierowca.data.model.BroadcastAuthBody
|
||||
import pl.firmatpp.kierowca.data.model.BroadcastAuthResponse
|
||||
import pl.firmatpp.kierowca.data.model.CancelLeaveRequestBody
|
||||
import pl.firmatpp.kierowca.data.model.CompleteRouteResponse
|
||||
import pl.firmatpp.kierowca.data.model.CreateLeaveRequestBody
|
||||
import pl.firmatpp.kierowca.data.model.DispatchSheetUploadResponse
|
||||
import pl.firmatpp.kierowca.data.model.LeaveRequestListResponse
|
||||
import pl.firmatpp.kierowca.data.model.LeaveRequestResponse
|
||||
import pl.firmatpp.kierowca.data.model.LeaveRequestTypesResponse
|
||||
import pl.firmatpp.kierowca.data.model.OtpResponse
|
||||
import pl.firmatpp.kierowca.data.model.NotificationPreferencesBody
|
||||
import pl.firmatpp.kierowca.data.model.NotificationPreferencesDto
|
||||
@@ -56,6 +61,35 @@ interface MobileDriverApi {
|
||||
@Query("date") date: String?,
|
||||
): BootstrapResponse
|
||||
|
||||
@GET("mobile/driver/leave-request-types")
|
||||
suspend fun leaveRequestTypes(
|
||||
@Header("Authorization") authorization: String,
|
||||
): LeaveRequestTypesResponse
|
||||
|
||||
@GET("mobile/driver/leave-requests")
|
||||
suspend fun leaveRequests(
|
||||
@Header("Authorization") authorization: String,
|
||||
): LeaveRequestListResponse
|
||||
|
||||
@POST("mobile/driver/leave-requests")
|
||||
suspend fun createLeaveRequest(
|
||||
@Header("Authorization") authorization: String,
|
||||
@Body body: CreateLeaveRequestBody,
|
||||
): LeaveRequestResponse
|
||||
|
||||
@GET("mobile/driver/leave-requests/{id}")
|
||||
suspend fun leaveRequest(
|
||||
@Header("Authorization") authorization: String,
|
||||
@Path("id") id: String,
|
||||
): LeaveRequestResponse
|
||||
|
||||
@POST("mobile/driver/leave-requests/{id}/cancel")
|
||||
suspend fun cancelLeaveRequest(
|
||||
@Header("Authorization") authorization: String,
|
||||
@Path("id") id: String,
|
||||
@Body body: CancelLeaveRequestBody,
|
||||
): LeaveRequestResponse
|
||||
|
||||
@GET("mobile/driver/routes/{routeId}")
|
||||
suspend fun route(
|
||||
@Header("Authorization") authorization: String,
|
||||
|
||||
@@ -65,6 +65,12 @@ data class DriverAppSettingsDto(
|
||||
val dispatchSheetRemindersEnabled: Boolean? = null,
|
||||
val dispatchSheetOnFridays: Boolean? = null,
|
||||
val dispatchSheetOnLastWorkingDay: Boolean? = null,
|
||||
val leaveRequests: LeaveRequestsConfigDto? = null,
|
||||
)
|
||||
|
||||
data class LeaveRequestsConfigDto(
|
||||
val enabled: Boolean = false,
|
||||
val types: List<String> = listOf("URLOP"),
|
||||
)
|
||||
|
||||
data class DispatchSheetReminderDto(
|
||||
@@ -157,6 +163,58 @@ data class NotificationPreferencesBody(
|
||||
val notifyNewRoutes: Boolean,
|
||||
)
|
||||
|
||||
data class LeaveRequestListResponse(
|
||||
val data: List<DriverLeaveRequestDto> = emptyList(),
|
||||
)
|
||||
|
||||
data class LeaveRequestResponse(
|
||||
val data: DriverLeaveRequestDto,
|
||||
)
|
||||
|
||||
data class LeaveRequestTypesResponse(
|
||||
val data: List<LeaveRequestTypeDto> = emptyList(),
|
||||
)
|
||||
|
||||
data class LeaveRequestTypeDto(
|
||||
val value: String,
|
||||
val label: String,
|
||||
)
|
||||
|
||||
data class CreateLeaveRequestBody(
|
||||
val dateFrom: String,
|
||||
val dateTo: String,
|
||||
val type: String,
|
||||
val note: String?,
|
||||
)
|
||||
|
||||
data class CancelLeaveRequestBody(
|
||||
val comment: String? = null,
|
||||
)
|
||||
|
||||
data class DriverLeaveRequestDto(
|
||||
val id: String,
|
||||
val driver: DriverDto? = null,
|
||||
val dateFrom: String?,
|
||||
val dateTo: String?,
|
||||
val type: String,
|
||||
val typeLabel: String? = null,
|
||||
val note: String? = null,
|
||||
val status: String,
|
||||
val submittedAt: String? = null,
|
||||
val decidedAt: String? = null,
|
||||
val decisionComment: String? = null,
|
||||
val events: List<DriverLeaveRequestEventDto> = emptyList(),
|
||||
)
|
||||
|
||||
data class DriverLeaveRequestEventDto(
|
||||
val id: String,
|
||||
val action: String,
|
||||
val fromStatus: String? = null,
|
||||
val toStatus: String? = null,
|
||||
val comment: String? = null,
|
||||
val createdAt: String? = null,
|
||||
)
|
||||
|
||||
data class RealtimeStatusBody(
|
||||
val reverbStatus: String,
|
||||
val socketId: String? = null,
|
||||
|
||||
@@ -151,5 +151,6 @@ class DriverSyncRepository(
|
||||
const val SCOPE_ROUTE_DETAIL = "route_detail"
|
||||
const val SCOPE_SETTINGS = "settings"
|
||||
const val SCOPE_DISPATCH_SHEET = "dispatch_sheet"
|
||||
const val SCOPE_LEAVE_REQUESTS = "leave_requests"
|
||||
}
|
||||
}
|
||||
|
||||
@@ -8,6 +8,7 @@ import java.io.File
|
||||
import pl.firmatpp.kierowca.data.ApiErrorKind
|
||||
import pl.firmatpp.kierowca.data.ApiErrorMapper
|
||||
import pl.firmatpp.kierowca.data.DriverRepository
|
||||
import pl.firmatpp.kierowca.diagnostics.AppDiagnostics
|
||||
|
||||
class DispatchSheetUploadWorker(
|
||||
appContext: Context,
|
||||
@@ -64,6 +65,17 @@ class DispatchSheetUploadWorker(
|
||||
Result.success()
|
||||
}.getOrElse { throwable ->
|
||||
val error = ApiErrorMapper.map(throwable)
|
||||
AppDiagnostics.reportNonFatal(
|
||||
throwable = throwable,
|
||||
operation = "dispatch_sheet_upload_worker",
|
||||
keys = mapOf(
|
||||
"client_request_id" to clientRequestId,
|
||||
"api_error_kind" to error.kind.name,
|
||||
"api_error_code" to error.code,
|
||||
"api_status_code" to error.statusCode,
|
||||
"retryable" to error.retryable,
|
||||
),
|
||||
)
|
||||
val status = if (error.retryable) PhotoUploadStatus.FailedRetryable else PhotoUploadStatus.FailedPermanent
|
||||
dao.updateStatus(
|
||||
clientRequestId = clientRequestId,
|
||||
|
||||
@@ -8,6 +8,7 @@ import java.io.File
|
||||
import pl.firmatpp.kierowca.data.ApiErrorKind
|
||||
import pl.firmatpp.kierowca.data.ApiErrorMapper
|
||||
import pl.firmatpp.kierowca.data.DriverRepository
|
||||
import pl.firmatpp.kierowca.diagnostics.AppDiagnostics
|
||||
|
||||
class PhotoUploadWorker(
|
||||
appContext: Context,
|
||||
@@ -64,6 +65,18 @@ class PhotoUploadWorker(
|
||||
Result.success()
|
||||
}.getOrElse { throwable ->
|
||||
val error = ApiErrorMapper.map(throwable)
|
||||
AppDiagnostics.reportNonFatal(
|
||||
throwable = throwable,
|
||||
operation = "photo_upload_worker",
|
||||
keys = mapOf(
|
||||
"client_request_id" to clientRequestId,
|
||||
"route_id" to upload.routeId,
|
||||
"api_error_kind" to error.kind.name,
|
||||
"api_error_code" to error.code,
|
||||
"api_status_code" to error.statusCode,
|
||||
"retryable" to error.retryable,
|
||||
),
|
||||
)
|
||||
val status = if (error.retryable) PhotoUploadStatus.FailedRetryable else PhotoUploadStatus.FailedPermanent
|
||||
dao.updateStatus(
|
||||
clientRequestId = clientRequestId,
|
||||
|
||||
@@ -0,0 +1,75 @@
|
||||
package pl.firmatpp.kierowca.diagnostics
|
||||
|
||||
import com.google.firebase.crashlytics.FirebaseCrashlytics
|
||||
|
||||
interface DiagnosticsSink {
|
||||
fun log(message: String)
|
||||
fun setUserId(userId: String)
|
||||
fun setCustomKey(key: String, value: String)
|
||||
fun recordException(throwable: Throwable)
|
||||
}
|
||||
|
||||
object NoOpDiagnosticsSink : DiagnosticsSink {
|
||||
override fun log(message: String) = Unit
|
||||
override fun setUserId(userId: String) = Unit
|
||||
override fun setCustomKey(key: String, value: String) = Unit
|
||||
override fun recordException(throwable: Throwable) = Unit
|
||||
}
|
||||
|
||||
class FirebaseCrashlyticsDiagnosticsSink(
|
||||
private val crashlytics: FirebaseCrashlytics = FirebaseCrashlytics.getInstance(),
|
||||
) : DiagnosticsSink {
|
||||
override fun log(message: String) {
|
||||
crashlytics.log(message)
|
||||
}
|
||||
|
||||
override fun setUserId(userId: String) {
|
||||
crashlytics.setUserId(userId)
|
||||
}
|
||||
|
||||
override fun setCustomKey(key: String, value: String) {
|
||||
crashlytics.setCustomKey(key, value)
|
||||
}
|
||||
|
||||
override fun recordException(throwable: Throwable) {
|
||||
crashlytics.recordException(throwable)
|
||||
}
|
||||
}
|
||||
|
||||
object AppDiagnostics {
|
||||
@Volatile
|
||||
private var sink: DiagnosticsSink = NoOpDiagnosticsSink
|
||||
|
||||
fun installFirebaseCrashlytics() {
|
||||
installSink(FirebaseCrashlyticsDiagnosticsSink())
|
||||
}
|
||||
|
||||
fun installSink(nextSink: DiagnosticsSink) {
|
||||
sink = nextSink
|
||||
}
|
||||
|
||||
fun setDriverId(driverId: String) {
|
||||
sink.setUserId(driverId)
|
||||
}
|
||||
|
||||
fun clearDriverId() {
|
||||
sink.setUserId("")
|
||||
}
|
||||
|
||||
fun log(message: String) {
|
||||
sink.log(message)
|
||||
}
|
||||
|
||||
fun reportNonFatal(
|
||||
throwable: Throwable,
|
||||
operation: String,
|
||||
keys: Map<String, Any?> = emptyMap(),
|
||||
) {
|
||||
sink.setCustomKey("operation", operation)
|
||||
keys.forEach { (key, value) ->
|
||||
sink.setCustomKey(key, value?.toString().orEmpty())
|
||||
}
|
||||
sink.log("non_fatal: $operation: ${throwable.message ?: throwable::class.java.simpleName}")
|
||||
sink.recordException(throwable)
|
||||
}
|
||||
}
|
||||
@@ -31,6 +31,21 @@ class DriverFirebaseMessagingService : FirebaseMessagingService() {
|
||||
return
|
||||
}
|
||||
|
||||
if (data["type"] == "driver_leave_request_decision") {
|
||||
LeaveRequestDecisionNotificationWorker.enqueue(
|
||||
context = applicationContext,
|
||||
leaveRequestId = data["leaveRequestId"],
|
||||
title = data["title"],
|
||||
body = data["body"],
|
||||
)
|
||||
DriverSyncWorker.enqueue(
|
||||
context = applicationContext,
|
||||
date = data["date"],
|
||||
routeId = data["routeId"],
|
||||
)
|
||||
return
|
||||
}
|
||||
|
||||
if (data["type"] != "driver_sync_hint") return
|
||||
|
||||
DriverSyncWorker.enqueue(
|
||||
|
||||
@@ -3,10 +3,10 @@ package pl.firmatpp.kierowca.sync
|
||||
import com.google.gson.Gson
|
||||
import com.google.gson.JsonObject
|
||||
import com.google.gson.JsonParser
|
||||
import java.util.concurrent.atomic.AtomicBoolean
|
||||
import kotlinx.coroutines.Job
|
||||
import java.util.concurrent.TimeUnit
|
||||
import kotlinx.coroutines.CoroutineScope
|
||||
import kotlinx.coroutines.Dispatchers
|
||||
import kotlinx.coroutines.Job
|
||||
import kotlinx.coroutines.SupervisorJob
|
||||
import kotlinx.coroutines.cancel
|
||||
import kotlinx.coroutines.delay
|
||||
@@ -16,73 +16,155 @@ import okhttp3.Request
|
||||
import okhttp3.WebSocket
|
||||
import okhttp3.WebSocketListener
|
||||
import pl.firmatpp.kierowca.data.DriverRepository
|
||||
import pl.firmatpp.kierowca.data.model.BroadcastAuthResponse
|
||||
import pl.firmatpp.kierowca.data.model.RealtimeConfigDto
|
||||
import pl.firmatpp.kierowca.diagnostics.AppDiagnostics
|
||||
|
||||
class DriverLiveSyncClient(
|
||||
private val repository: DriverRepository,
|
||||
private val onConnected: () -> Unit,
|
||||
private val onHint: (DriverSyncHint) -> Unit,
|
||||
private val client: OkHttpClient = OkHttpClient(),
|
||||
private val gson: Gson = Gson(),
|
||||
) {
|
||||
private companion object {
|
||||
const val HEARTBEAT_INTERVAL_MS = 10_000L
|
||||
interface DriverLiveSyncGateway {
|
||||
suspend fun broadcastAuth(socketId: String, channelName: String): BroadcastAuthResponse
|
||||
suspend fun storeRealtimeStatus(status: String, socketId: String?, error: String?)
|
||||
}
|
||||
|
||||
private val scope = CoroutineScope(SupervisorJob() + Dispatchers.IO)
|
||||
private val started = AtomicBoolean(false)
|
||||
class DriverRepositoryLiveSyncGateway(
|
||||
private val repository: DriverRepository,
|
||||
) : DriverLiveSyncGateway {
|
||||
override suspend fun broadcastAuth(socketId: String, channelName: String): BroadcastAuthResponse =
|
||||
repository.broadcastAuth(socketId, channelName)
|
||||
|
||||
override suspend fun storeRealtimeStatus(status: String, socketId: String?, error: String?) {
|
||||
repository.storeRealtimeStatus(status, socketId, error)
|
||||
}
|
||||
}
|
||||
|
||||
interface LiveWebSocketFactory {
|
||||
fun newWebSocket(url: String, listener: WebSocketListener): WebSocket
|
||||
}
|
||||
|
||||
class OkHttpLiveWebSocketFactory(
|
||||
private val client: OkHttpClient,
|
||||
) : LiveWebSocketFactory {
|
||||
override fun newWebSocket(url: String, listener: WebSocketListener): WebSocket =
|
||||
client.newWebSocket(Request.Builder().url(url).build(), listener)
|
||||
}
|
||||
|
||||
private enum class LiveSyncConnectionState {
|
||||
Stopped,
|
||||
WaitingForNetwork,
|
||||
Disconnected,
|
||||
Connecting,
|
||||
Subscribing,
|
||||
Connected,
|
||||
}
|
||||
|
||||
class DriverLiveSyncClient(
|
||||
private val gateway: DriverLiveSyncGateway,
|
||||
private val onConnected: () -> Unit,
|
||||
private val onHint: (DriverSyncHint) -> Unit,
|
||||
private val webSocketFactory: LiveWebSocketFactory = OkHttpLiveWebSocketFactory(defaultOkHttpClient()),
|
||||
private val gson: Gson = Gson(),
|
||||
private val scope: CoroutineScope = CoroutineScope(SupervisorJob() + Dispatchers.IO),
|
||||
private val reconnectDelaysMs: List<Long> = DEFAULT_RECONNECT_DELAYS_MS,
|
||||
private val staleTimeoutMs: Long = STALE_TIMEOUT_MS,
|
||||
) {
|
||||
constructor(
|
||||
repository: DriverRepository,
|
||||
onConnected: () -> Unit,
|
||||
onHint: (DriverSyncHint) -> Unit,
|
||||
client: OkHttpClient = defaultOkHttpClient(),
|
||||
gson: Gson = Gson(),
|
||||
) : this(
|
||||
gateway = DriverRepositoryLiveSyncGateway(repository),
|
||||
onConnected = onConnected,
|
||||
onHint = onHint,
|
||||
webSocketFactory = OkHttpLiveWebSocketFactory(client),
|
||||
gson = gson,
|
||||
)
|
||||
|
||||
private companion object {
|
||||
const val HEARTBEAT_INTERVAL_MS = 10_000L
|
||||
const val STALE_TIMEOUT_MS = 30_000L
|
||||
val DEFAULT_RECONNECT_DELAYS_MS = listOf(1_000L, 2_000L, 5_000L, 10_000L, 30_000L)
|
||||
|
||||
fun defaultOkHttpClient(): OkHttpClient =
|
||||
OkHttpClient.Builder()
|
||||
.pingInterval(15, TimeUnit.SECONDS)
|
||||
.build()
|
||||
}
|
||||
|
||||
private val lock = Any()
|
||||
private var desiredActive: Boolean = false
|
||||
private var networkAvailable: Boolean = true
|
||||
private var state: LiveSyncConnectionState = LiveSyncConnectionState.Stopped
|
||||
private var webSocket: WebSocket? = null
|
||||
private var driverId: String? = null
|
||||
private var realtimeConfig: RealtimeConfigDto? = null
|
||||
private var socketId: String? = null
|
||||
private var reconnectJob: Job? = null
|
||||
private var heartbeatJob: Job? = null
|
||||
private var staleWatchdogJob: Job? = null
|
||||
private var reconnectAttempt: Int = 0
|
||||
private var messageVersion: Long = 0
|
||||
|
||||
fun start(driverId: String, config: RealtimeConfigDto?) {
|
||||
val appKey = config?.reverbAppKey?.takeIf { it.isNotBlank() } ?: return
|
||||
val wsBaseUrl = config.reverbWsBaseUrl?.takeIf { it.isNotBlank() } ?: return
|
||||
if (!config.reverbEnabled) return
|
||||
if (!isConfigUsable(config)) return
|
||||
|
||||
synchronized(lock) {
|
||||
realtimeConfig = config
|
||||
this.driverId = driverId
|
||||
if (!started.compareAndSet(false, true)) return
|
||||
|
||||
val wsUrl = wsBaseUrl.trimEnd('/') +
|
||||
"/" + appKey +
|
||||
"?protocol=7&client=android&version=1.0&flash=false"
|
||||
|
||||
webSocket = client.newWebSocket(
|
||||
Request.Builder().url(wsUrl).build(),
|
||||
object : WebSocketListener() {
|
||||
override fun onMessage(webSocket: WebSocket, text: String) {
|
||||
handleMessage(webSocket, text)
|
||||
desiredActive = true
|
||||
}
|
||||
ensureConnected()
|
||||
}
|
||||
|
||||
override fun onClosed(webSocket: WebSocket, code: Int, reason: String) {
|
||||
stopHeartbeat()
|
||||
reportRealtimeStatus("disconnected", reason.takeIf { it.isNotBlank() })
|
||||
started.set(false)
|
||||
scheduleReconnect()
|
||||
}
|
||||
|
||||
override fun onFailure(webSocket: WebSocket, t: Throwable, response: okhttp3.Response?) {
|
||||
stopHeartbeat()
|
||||
reportRealtimeStatus("error", t.message ?: response?.message)
|
||||
started.set(false)
|
||||
scheduleReconnect()
|
||||
}
|
||||
},
|
||||
fun ensureConnected() {
|
||||
val shouldConnect = synchronized(lock) {
|
||||
desiredActive &&
|
||||
networkAvailable &&
|
||||
state !in setOf(
|
||||
LiveSyncConnectionState.Connecting,
|
||||
LiveSyncConnectionState.Subscribing,
|
||||
LiveSyncConnectionState.Connected,
|
||||
)
|
||||
}
|
||||
if (shouldConnect) connectNow()
|
||||
}
|
||||
|
||||
fun onNetworkAvailable(available: Boolean) {
|
||||
val shouldReconnect = synchronized(lock) {
|
||||
networkAvailable = available
|
||||
if (!available) {
|
||||
reconnectJob?.cancel()
|
||||
reconnectJob = null
|
||||
closeSocketLocked("network_lost")
|
||||
state = if (desiredActive) LiveSyncConnectionState.WaitingForNetwork else LiveSyncConnectionState.Stopped
|
||||
false
|
||||
} else {
|
||||
desiredActive && state == LiveSyncConnectionState.WaitingForNetwork
|
||||
}
|
||||
}
|
||||
|
||||
if (available) {
|
||||
reportRealtimeStatus("reconnecting", "network_available")
|
||||
} else {
|
||||
reportRealtimeStatus("disconnected", "network_lost")
|
||||
}
|
||||
|
||||
if (shouldReconnect) connectNow(resetAttempt = true)
|
||||
}
|
||||
|
||||
fun stop() {
|
||||
started.set(false)
|
||||
stopHeartbeat()
|
||||
reportRealtimeStatus("disconnected", "client_stop")
|
||||
webSocket?.close(1000, "logout")
|
||||
webSocket = null
|
||||
synchronized(lock) {
|
||||
desiredActive = false
|
||||
reconnectJob?.cancel()
|
||||
reconnectJob = null
|
||||
closeSocketLocked("client_stop")
|
||||
driverId = null
|
||||
realtimeConfig = null
|
||||
socketId = null
|
||||
reconnectAttempt = 0
|
||||
state = LiveSyncConnectionState.Stopped
|
||||
}
|
||||
reportRealtimeStatus("disconnected", "client_stop")
|
||||
}
|
||||
|
||||
fun close() {
|
||||
@@ -90,35 +172,112 @@ class DriverLiveSyncClient(
|
||||
scope.cancel()
|
||||
}
|
||||
|
||||
private fun connectNow(resetAttempt: Boolean = false) {
|
||||
val wsUrl = synchronized(lock) {
|
||||
val config = realtimeConfig ?: return
|
||||
val id = driverId ?: return
|
||||
if (!desiredActive || !networkAvailable || !isConfigUsable(config)) return
|
||||
if (state == LiveSyncConnectionState.Connecting || state == LiveSyncConnectionState.Subscribing || state == LiveSyncConnectionState.Connected) return
|
||||
|
||||
if (resetAttempt) reconnectAttempt = 0
|
||||
reconnectJob?.cancel()
|
||||
reconnectJob = null
|
||||
socketId = null
|
||||
state = LiveSyncConnectionState.Connecting
|
||||
|
||||
val appKey = config.reverbAppKey.orEmpty()
|
||||
val wsBaseUrl = config.reverbWsBaseUrl.orEmpty()
|
||||
wsBaseUrl.trimEnd('/') + "/" + appKey + "?protocol=7&client=android&version=1.0&flash=false"
|
||||
}
|
||||
|
||||
val socket = webSocketFactory.newWebSocket(
|
||||
wsUrl,
|
||||
object : WebSocketListener() {
|
||||
override fun onMessage(webSocket: WebSocket, text: String) {
|
||||
handleMessage(webSocket, text)
|
||||
}
|
||||
|
||||
override fun onClosed(webSocket: WebSocket, code: Int, reason: String) {
|
||||
handleDisconnect(
|
||||
status = "disconnected",
|
||||
reason = reason.takeIf { it.isNotBlank() } ?: "closed",
|
||||
socket = webSocket,
|
||||
)
|
||||
}
|
||||
|
||||
override fun onFailure(webSocket: WebSocket, t: Throwable, response: okhttp3.Response?) {
|
||||
AppDiagnostics.log("realtime_error: ${t.message ?: response?.message ?: "unknown"}")
|
||||
handleDisconnect(
|
||||
status = "error",
|
||||
reason = t.message ?: response?.message ?: "failure",
|
||||
socket = webSocket,
|
||||
)
|
||||
}
|
||||
},
|
||||
)
|
||||
|
||||
synchronized(lock) {
|
||||
webSocket = socket
|
||||
}
|
||||
}
|
||||
|
||||
private fun handleMessage(socket: WebSocket, text: String) {
|
||||
val isCurrentSocket = synchronized(lock) {
|
||||
if (webSocket !== socket) {
|
||||
false
|
||||
} else {
|
||||
messageVersion += 1
|
||||
true
|
||||
}
|
||||
}
|
||||
if (!isCurrentSocket) return
|
||||
|
||||
val root = runCatching { JsonParser.parseString(text).asJsonObject }.getOrNull() ?: return
|
||||
val event = root.string("event") ?: return
|
||||
|
||||
when (event) {
|
||||
"pusher:ping" -> {
|
||||
socket.send("""{"event":"pusher:pong","data":{}}""")
|
||||
restartStaleWatchdogIfConnected()
|
||||
}
|
||||
"pusher:connection_established" -> {
|
||||
socketId = root.dataObject()?.string("socket_id") ?: return
|
||||
subscribe(socket, socketId ?: return)
|
||||
val nextSocketId = root.dataObject()?.string("socket_id") ?: return
|
||||
synchronized(lock) {
|
||||
socketId = nextSocketId
|
||||
state = LiveSyncConnectionState.Subscribing
|
||||
}
|
||||
subscribe(socket, nextSocketId)
|
||||
}
|
||||
"pusher_internal:subscription_succeeded" -> {
|
||||
val channel = root.string("channel")
|
||||
val id = driverId ?: return
|
||||
val id = synchronized(lock) { driverId } ?: return
|
||||
if (channel == "private-driver-mobile.$id") {
|
||||
synchronized(lock) {
|
||||
reconnectAttempt = 0
|
||||
state = LiveSyncConnectionState.Connected
|
||||
}
|
||||
reportRealtimeStatus("connected")
|
||||
startHeartbeat()
|
||||
startStaleWatchdog()
|
||||
onConnected()
|
||||
}
|
||||
}
|
||||
"DriverMobileSyncHint" -> parseHint(root.dataObject())?.let(onHint)
|
||||
"DriverMobileSyncHint" -> {
|
||||
restartStaleWatchdogIfConnected()
|
||||
parseHint(root.dataObject())?.let(onHint)
|
||||
}
|
||||
else -> restartStaleWatchdogIfConnected()
|
||||
}
|
||||
}
|
||||
|
||||
private fun subscribe(socket: WebSocket, socketId: String) {
|
||||
val id = driverId ?: return
|
||||
val id = synchronized(lock) { driverId } ?: return
|
||||
val channel = "private-driver-mobile.$id"
|
||||
|
||||
scope.launch {
|
||||
runCatching {
|
||||
val auth = repository.broadcastAuth(socketId, channel).auth
|
||||
val auth = gateway.broadcastAuth(socketId, channel).auth
|
||||
if (!isCurrentSocket(socket)) return@launch
|
||||
val payload = mapOf(
|
||||
"event" to "pusher:subscribe",
|
||||
"data" to mapOf(
|
||||
@@ -126,11 +285,129 @@ class DriverLiveSyncClient(
|
||||
"auth" to auth,
|
||||
),
|
||||
)
|
||||
socket.send(gson.toJson(payload))
|
||||
check(socket.send(gson.toJson(payload))) { "WebSocket send returned false" }
|
||||
}.onFailure { throwable ->
|
||||
AppDiagnostics.log("realtime_subscription_error: ${throwable.message ?: throwable::class.java.simpleName}")
|
||||
socket.close(1000, "subscription_failed")
|
||||
handleDisconnect("error", "subscription_failed", socket)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private fun handleDisconnect(status: String, reason: String?, socket: WebSocket? = null) {
|
||||
val shouldSchedule = synchronized(lock) {
|
||||
if (socket != null && webSocket !== socket) return
|
||||
|
||||
stopHeartbeatLocked()
|
||||
stopStaleWatchdogLocked()
|
||||
webSocket = null
|
||||
socketId = null
|
||||
|
||||
if (!desiredActive) {
|
||||
state = LiveSyncConnectionState.Stopped
|
||||
false
|
||||
} else if (!networkAvailable) {
|
||||
state = LiveSyncConnectionState.WaitingForNetwork
|
||||
false
|
||||
} else {
|
||||
state = LiveSyncConnectionState.Disconnected
|
||||
true
|
||||
}
|
||||
}
|
||||
|
||||
reportRealtimeStatus(status, reason)
|
||||
if (shouldSchedule) scheduleReconnect()
|
||||
}
|
||||
|
||||
private fun scheduleReconnect() {
|
||||
val delayMs = synchronized(lock) {
|
||||
if (!desiredActive || !networkAvailable || state == LiveSyncConnectionState.Stopped) return
|
||||
if (reconnectJob?.isActive == true) return
|
||||
val delay = reconnectDelaysMs.getOrElse(reconnectAttempt) { reconnectDelaysMs.last() }
|
||||
reconnectAttempt += 1
|
||||
delay
|
||||
}
|
||||
|
||||
reconnectJob = scope.launch {
|
||||
delay(delayMs)
|
||||
connectNow()
|
||||
}
|
||||
}
|
||||
|
||||
private fun startHeartbeat() {
|
||||
synchronized(lock) {
|
||||
heartbeatJob?.cancel()
|
||||
heartbeatJob = scope.launch {
|
||||
while (true) {
|
||||
reportRealtimeStatus("connected")
|
||||
delay(HEARTBEAT_INTERVAL_MS)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private fun startStaleWatchdog() {
|
||||
synchronized(lock) {
|
||||
stopStaleWatchdogLocked()
|
||||
val observedVersion = messageVersion
|
||||
staleWatchdogJob = scope.launch {
|
||||
delay(staleTimeoutMs)
|
||||
val socketToClose = synchronized(lock) {
|
||||
if (state == LiveSyncConnectionState.Connected && messageVersion == observedVersion) {
|
||||
webSocket
|
||||
} else {
|
||||
null
|
||||
}
|
||||
}
|
||||
|
||||
if (socketToClose != null) {
|
||||
socketToClose.close(1001, "stale_connection")
|
||||
handleDisconnect("error", "stale_connection", socketToClose)
|
||||
ensureConnected()
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private fun restartStaleWatchdogIfConnected() {
|
||||
val connected = synchronized(lock) { state == LiveSyncConnectionState.Connected }
|
||||
if (connected) startStaleWatchdog()
|
||||
}
|
||||
|
||||
private fun isCurrentSocket(socket: WebSocket): Boolean =
|
||||
synchronized(lock) { webSocket === socket }
|
||||
|
||||
private fun closeSocketLocked(reason: String) {
|
||||
stopHeartbeatLocked()
|
||||
stopStaleWatchdogLocked()
|
||||
webSocket?.close(1000, reason)
|
||||
webSocket = null
|
||||
socketId = null
|
||||
}
|
||||
|
||||
private fun stopHeartbeatLocked() {
|
||||
heartbeatJob?.cancel()
|
||||
heartbeatJob = null
|
||||
}
|
||||
|
||||
private fun stopStaleWatchdogLocked() {
|
||||
staleWatchdogJob?.cancel()
|
||||
staleWatchdogJob = null
|
||||
}
|
||||
|
||||
private fun reportRealtimeStatus(status: String, error: String? = null) {
|
||||
scope.launch {
|
||||
runCatching {
|
||||
gateway.storeRealtimeStatus(status, socketId, error)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private fun isConfigUsable(config: RealtimeConfigDto?): Boolean =
|
||||
config?.reverbEnabled == true &&
|
||||
!config.reverbAppKey.isNullOrBlank() &&
|
||||
!config.reverbWsBaseUrl.isNullOrBlank()
|
||||
|
||||
private fun parseHint(data: JsonObject?): DriverSyncHint? {
|
||||
if (data == null || data.string("type") != "driver_sync_hint") return null
|
||||
|
||||
@@ -150,37 +427,4 @@ class DriverLiveSyncClient(
|
||||
|
||||
private fun JsonObject.string(name: String): String? =
|
||||
get(name)?.takeIf { !it.isJsonNull }?.asString
|
||||
|
||||
private fun scheduleReconnect() {
|
||||
val id = driverId ?: return
|
||||
scope.launch {
|
||||
delay(5_000)
|
||||
if (!started.get() && driverId == id) {
|
||||
start(id, realtimeConfig)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private fun startHeartbeat() {
|
||||
heartbeatJob?.cancel()
|
||||
heartbeatJob = scope.launch {
|
||||
while (started.get()) {
|
||||
reportRealtimeStatus("connected")
|
||||
delay(HEARTBEAT_INTERVAL_MS)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private fun stopHeartbeat() {
|
||||
heartbeatJob?.cancel()
|
||||
heartbeatJob = null
|
||||
}
|
||||
|
||||
private fun reportRealtimeStatus(status: String, error: String? = null) {
|
||||
scope.launch {
|
||||
runCatching {
|
||||
repository.storeRealtimeStatus(status, socketId, error)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -14,6 +14,7 @@ import java.util.concurrent.TimeUnit
|
||||
import pl.firmatpp.kierowca.data.ApiErrorKind
|
||||
import pl.firmatpp.kierowca.data.ApiErrorMapper
|
||||
import pl.firmatpp.kierowca.data.sync.DriverSyncRepository
|
||||
import pl.firmatpp.kierowca.diagnostics.AppDiagnostics
|
||||
|
||||
class DriverSyncWorker(
|
||||
appContext: Context,
|
||||
@@ -47,6 +48,10 @@ class DriverSyncWorker(
|
||||
syncRepository.bootstrap(scope.date ?: date)
|
||||
refreshed = true
|
||||
}
|
||||
DriverSyncRepository.SCOPE_LEAVE_REQUESTS -> {
|
||||
syncRepository.saveSyncStates(response)
|
||||
refreshed = true
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -58,6 +63,18 @@ class DriverSyncWorker(
|
||||
Result.success()
|
||||
}.getOrElse { throwable ->
|
||||
val error = ApiErrorMapper.map(throwable)
|
||||
AppDiagnostics.reportNonFatal(
|
||||
throwable = throwable,
|
||||
operation = "driver_sync_worker",
|
||||
keys = mapOf(
|
||||
"date" to inputData.getString(KEY_DATE),
|
||||
"route_id" to inputData.getString(KEY_ROUTE_ID),
|
||||
"api_error_kind" to error.kind.name,
|
||||
"api_error_code" to error.code,
|
||||
"api_status_code" to error.statusCode,
|
||||
"retryable" to error.retryable,
|
||||
),
|
||||
)
|
||||
if (error.retryable && error.kind != ApiErrorKind.Auth) Result.retry() else Result.failure()
|
||||
}
|
||||
|
||||
|
||||
+110
@@ -0,0 +1,110 @@
|
||||
package pl.firmatpp.kierowca.sync
|
||||
|
||||
import android.Manifest
|
||||
import android.app.NotificationChannel
|
||||
import android.app.NotificationManager
|
||||
import android.app.PendingIntent
|
||||
import android.content.Context
|
||||
import android.content.Intent
|
||||
import android.content.pm.PackageManager
|
||||
import android.os.Build
|
||||
import androidx.core.app.NotificationCompat
|
||||
import androidx.core.app.NotificationManagerCompat
|
||||
import androidx.core.content.ContextCompat
|
||||
import androidx.work.CoroutineWorker
|
||||
import androidx.work.ExistingWorkPolicy
|
||||
import androidx.work.OneTimeWorkRequestBuilder
|
||||
import androidx.work.WorkManager
|
||||
import androidx.work.WorkerParameters
|
||||
import androidx.work.workDataOf
|
||||
import pl.firmatpp.kierowca.MainActivity
|
||||
import pl.firmatpp.kierowca.R
|
||||
|
||||
class LeaveRequestDecisionNotificationWorker(
|
||||
appContext: Context,
|
||||
params: WorkerParameters,
|
||||
) : CoroutineWorker(appContext, params) {
|
||||
override suspend fun doWork(): Result {
|
||||
val leaveRequestId = inputData.getString(KEY_LEAVE_REQUEST_ID)?.takeIf(String::isNotBlank)
|
||||
?: return Result.success()
|
||||
|
||||
if (!canShowNotifications(applicationContext)) {
|
||||
return Result.success()
|
||||
}
|
||||
|
||||
createChannel(applicationContext)
|
||||
val title = inputData.getString(KEY_TITLE)?.takeIf(String::isNotBlank) ?: "Decyzja w sprawie urlopu"
|
||||
val body = inputData.getString(KEY_BODY)?.takeIf(String::isNotBlank) ?: "Status wniosku urlopowego został zmieniony."
|
||||
val intent = Intent(applicationContext, MainActivity::class.java).apply {
|
||||
flags = Intent.FLAG_ACTIVITY_CLEAR_TOP or Intent.FLAG_ACTIVITY_SINGLE_TOP
|
||||
putExtra(MainActivity.EXTRA_LEAVE_REQUEST_ID, leaveRequestId)
|
||||
}
|
||||
val pendingIntent = PendingIntent.getActivity(
|
||||
applicationContext,
|
||||
leaveRequestId.hashCode(),
|
||||
intent,
|
||||
PendingIntent.FLAG_UPDATE_CURRENT or PendingIntent.FLAG_IMMUTABLE,
|
||||
)
|
||||
|
||||
val notification = NotificationCompat.Builder(applicationContext, CHANNEL_ID)
|
||||
.setSmallIcon(R.mipmap.ic_launcher)
|
||||
.setContentTitle(title)
|
||||
.setContentText(body)
|
||||
.setStyle(NotificationCompat.BigTextStyle().bigText(body))
|
||||
.setContentIntent(pendingIntent)
|
||||
.setAutoCancel(true)
|
||||
.setCategory(NotificationCompat.CATEGORY_STATUS)
|
||||
.setPriority(NotificationCompat.PRIORITY_DEFAULT)
|
||||
.build()
|
||||
|
||||
NotificationManagerCompat.from(applicationContext).notify(leaveRequestId.hashCode(), notification)
|
||||
return Result.success()
|
||||
}
|
||||
|
||||
private fun createChannel(context: Context) {
|
||||
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.O) return
|
||||
|
||||
val channel = NotificationChannel(
|
||||
CHANNEL_ID,
|
||||
"Wnioski urlopowe",
|
||||
NotificationManager.IMPORTANCE_DEFAULT,
|
||||
).apply {
|
||||
description = "Powiadomienia o decyzjach w sprawie wniosków urlopowych."
|
||||
}
|
||||
context.getSystemService(NotificationManager::class.java)?.createNotificationChannel(channel)
|
||||
}
|
||||
|
||||
private fun canShowNotifications(context: Context): Boolean {
|
||||
val hasRuntimePermission = Build.VERSION.SDK_INT < Build.VERSION_CODES.TIRAMISU ||
|
||||
ContextCompat.checkSelfPermission(context, Manifest.permission.POST_NOTIFICATIONS) == PackageManager.PERMISSION_GRANTED
|
||||
|
||||
return hasRuntimePermission && NotificationManagerCompat.from(context).areNotificationsEnabled()
|
||||
}
|
||||
|
||||
companion object {
|
||||
private const val CHANNEL_ID = "leave_request_decisions"
|
||||
private const val KEY_LEAVE_REQUEST_ID = "leaveRequestId"
|
||||
private const val KEY_TITLE = "title"
|
||||
private const val KEY_BODY = "body"
|
||||
|
||||
fun enqueue(context: Context, leaveRequestId: String?, title: String?, body: String?) {
|
||||
if (leaveRequestId.isNullOrBlank()) return
|
||||
|
||||
val request = OneTimeWorkRequestBuilder<LeaveRequestDecisionNotificationWorker>()
|
||||
.setInputData(
|
||||
workDataOf(
|
||||
KEY_LEAVE_REQUEST_ID to leaveRequestId,
|
||||
KEY_TITLE to title,
|
||||
KEY_BODY to body,
|
||||
),
|
||||
)
|
||||
.build()
|
||||
|
||||
WorkManager.getInstance(context).enqueueUniqueWork(
|
||||
"driver-leave-request-decision-$leaveRequestId",
|
||||
ExistingWorkPolicy.REPLACE,
|
||||
request,
|
||||
)
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -26,6 +26,7 @@ import pl.firmatpp.kierowca.R
|
||||
import pl.firmatpp.kierowca.data.ApiErrorKind
|
||||
import pl.firmatpp.kierowca.data.ApiErrorMapper
|
||||
import pl.firmatpp.kierowca.data.DriverRepository
|
||||
import pl.firmatpp.kierowca.diagnostics.AppDiagnostics
|
||||
import retrofit2.HttpException
|
||||
|
||||
class NewRouteNotificationWorker(
|
||||
@@ -76,6 +77,17 @@ class NewRouteNotificationWorker(
|
||||
Result.success()
|
||||
} else {
|
||||
val error = ApiErrorMapper.map(throwable)
|
||||
AppDiagnostics.reportNonFatal(
|
||||
throwable = throwable,
|
||||
operation = "new_route_notification_worker",
|
||||
keys = mapOf(
|
||||
"route_id" to routeId,
|
||||
"api_error_kind" to error.kind.name,
|
||||
"api_error_code" to error.code,
|
||||
"api_status_code" to error.statusCode,
|
||||
"retryable" to error.retryable,
|
||||
),
|
||||
)
|
||||
if (error.retryable && error.kind != ApiErrorKind.Auth) Result.retry() else Result.success()
|
||||
}
|
||||
}
|
||||
|
||||
File diff suppressed because it is too large
Load Diff
@@ -0,0 +1,55 @@
|
||||
package pl.firmatpp.kierowca.ui
|
||||
|
||||
import java.time.LocalDate
|
||||
|
||||
data class LeaveRequestsConfig(
|
||||
val enabled: Boolean = false,
|
||||
val types: List<String> = listOf("URLOP"),
|
||||
)
|
||||
|
||||
object DriverLeaveRequestUiRules {
|
||||
fun isFeatureVisible(config: LeaveRequestsConfig?): Boolean =
|
||||
config?.enabled == true && config.types.isNotEmpty()
|
||||
|
||||
fun statusLabel(status: String): String =
|
||||
when (status) {
|
||||
"pending" -> "Oczekuje"
|
||||
"approved" -> "Zatwierdzony"
|
||||
"rejected" -> "Odrzucony"
|
||||
"cancel_requested" -> "Anulowanie do decyzji"
|
||||
"cancelled" -> "Anulowany"
|
||||
"revoked" -> "Cofnięto decyzję"
|
||||
else -> "Nieznany status"
|
||||
}
|
||||
|
||||
fun eventActionLabel(action: String): String =
|
||||
when (action) {
|
||||
"submitted" -> "Wniosek złożony"
|
||||
"approved" -> "Wniosek zatwierdzony"
|
||||
"rejected" -> "Wniosek odrzucony"
|
||||
"revoked" -> "Decyzja cofnięta"
|
||||
"cancelled_by_driver" -> "Wniosek anulowany przez kierowcę"
|
||||
"cancellation_requested" -> "Kierowca poprosił o anulowanie"
|
||||
"cancellation_approved" -> "Anulowanie zatwierdzone"
|
||||
"cancellation_rejected" -> "Anulowanie odrzucone"
|
||||
else -> "Aktualizacja wniosku"
|
||||
}
|
||||
|
||||
fun typeLabel(type: String): String =
|
||||
when (type) {
|
||||
"URLOP" -> "Urlop"
|
||||
"CHOROBOWE" -> "Chorobowe"
|
||||
"SZKOLENIE" -> "Szkolenie"
|
||||
"WOLNE" -> "Dzień wolny"
|
||||
"INNE" -> "Inne"
|
||||
else -> type
|
||||
}
|
||||
|
||||
fun canCancel(status: String, dateFrom: String, today: LocalDate = LocalDate.now()): Boolean {
|
||||
if (status == "pending") return true
|
||||
if (status != "approved") return false
|
||||
|
||||
val start = runCatching { LocalDate.parse(dateFrom) }.getOrNull() ?: return false
|
||||
return start.isAfter(today)
|
||||
}
|
||||
}
|
||||
@@ -59,6 +59,18 @@ private fun polishWeekdayPhrase(day: DayOfWeek): String =
|
||||
fun inlinePhotoGridRows(photoCount: Int): Int =
|
||||
if (photoCount <= 0) 0 else (photoCount + 1) / 2
|
||||
|
||||
fun shouldShowPhotoPreviewLoadingState(hasPhoto: Boolean, isLoading: Boolean, isError: Boolean): Boolean =
|
||||
hasPhoto && isLoading && !isError
|
||||
|
||||
fun shouldShowPhotoPreviewErrorState(hasPhoto: Boolean, isError: Boolean): Boolean =
|
||||
hasPhoto && isError
|
||||
|
||||
fun shouldShowPhotoTileLoadingState(hasPhoto: Boolean, isLoading: Boolean, isError: Boolean): Boolean =
|
||||
hasPhoto && isLoading && !isError
|
||||
|
||||
fun shouldShowPhotoTileErrorState(hasPhoto: Boolean, isError: Boolean): Boolean =
|
||||
hasPhoto && isError
|
||||
|
||||
fun authBrandBannerHeightDp(screenHeightDp: Int): Int =
|
||||
when {
|
||||
screenHeightDp < 640 -> 118
|
||||
@@ -152,6 +164,15 @@ fun dispatchSheetPrimaryActionLabel(reminder: DispatchSheetReminderDto?, hasQueu
|
||||
else -> "Zrób zdjęcie"
|
||||
}
|
||||
|
||||
fun offlineStaleBannerMessage(isOnline: Boolean, isStale: Boolean, syncLabel: String): String? =
|
||||
when {
|
||||
isOnline && !isStale -> null
|
||||
!isOnline && syncLabel == "brak zapisanej synchronizacji" ->
|
||||
"Brak połączenia z serwerem. Sprawdź internet i spróbuj ponownie."
|
||||
!isOnline -> "Brak połączenia z serwerem. Dane z $syncLabel mogą być nieaktualne."
|
||||
else -> "Dane mogą być nieaktualne. Ostatnia synchronizacja: $syncLabel."
|
||||
}
|
||||
|
||||
private fun parseIsoOffsetEpochMillis(value: String?): Long? =
|
||||
value?.takeIf { it.isNotBlank() }?.let {
|
||||
runCatching { OffsetDateTime.parse(it).toInstant().toEpochMilli() }.getOrNull()
|
||||
|
||||
@@ -11,6 +11,7 @@ import kotlinx.coroutines.flow.StateFlow
|
||||
import kotlinx.coroutines.flow.update
|
||||
import kotlinx.coroutines.launch
|
||||
import java.time.LocalDate
|
||||
import pl.firmatpp.kierowca.data.AppPreferencesStore
|
||||
import pl.firmatpp.kierowca.data.ApiErrorMapper
|
||||
import pl.firmatpp.kierowca.data.ApiErrorKind
|
||||
import pl.firmatpp.kierowca.data.DriverRepository
|
||||
@@ -19,17 +20,20 @@ import pl.firmatpp.kierowca.data.sync.DriverSyncRepository
|
||||
import pl.firmatpp.kierowca.data.sync.NetworkMonitor
|
||||
import pl.firmatpp.kierowca.data.model.DispatchSheetReminderDto
|
||||
import pl.firmatpp.kierowca.data.model.DriverDto
|
||||
import pl.firmatpp.kierowca.data.model.DriverLeaveRequestDto
|
||||
import pl.firmatpp.kierowca.data.model.DriverRouteDto
|
||||
import pl.firmatpp.kierowca.data.model.RoutePhotoDto
|
||||
import pl.firmatpp.kierowca.data.upload.DispatchSheetUploadEntity
|
||||
import pl.firmatpp.kierowca.data.upload.DispatchSheetUploadOutbox
|
||||
import pl.firmatpp.kierowca.data.upload.PhotoUploadEntity
|
||||
import pl.firmatpp.kierowca.data.upload.PhotoUploadOutbox
|
||||
import pl.firmatpp.kierowca.diagnostics.AppDiagnostics
|
||||
import pl.firmatpp.kierowca.sync.DriverLiveSyncClient
|
||||
import pl.firmatpp.kierowca.sync.DriverSyncHint
|
||||
import pl.firmatpp.kierowca.sync.DriverSyncWorker
|
||||
import pl.firmatpp.kierowca.ui.theme.AppThemeMode
|
||||
|
||||
enum class DriverScreen { Initializing, Phone, Otp, Routes, Profile, Detail, Photo, PhotoQueue }
|
||||
enum class DriverScreen { Initializing, Phone, Otp, Routes, Profile, Detail, Photo, PhotoQueue, LeaveRequests, LeaveRequestDetail, AddLeaveRequest }
|
||||
|
||||
data class DriverUiState(
|
||||
val screen: DriverScreen = DriverScreen.Initializing,
|
||||
@@ -55,6 +59,13 @@ data class DriverUiState(
|
||||
val queuedPhotoUploads: List<PhotoUploadEntity> = emptyList(),
|
||||
val dispatchSheetReminder: DispatchSheetReminderDto? = null,
|
||||
val dispatchSheetUploads: List<DispatchSheetUploadEntity> = emptyList(),
|
||||
val leaveRequestsConfig: LeaveRequestsConfig? = null,
|
||||
val leaveRequests: List<DriverLeaveRequestDto> = emptyList(),
|
||||
val selectedLeaveRequest: DriverLeaveRequestDto? = null,
|
||||
val leaveRequestDateFrom: String = LocalDate.now().toString(),
|
||||
val leaveRequestDateTo: String = LocalDate.now().toString(),
|
||||
val leaveRequestType: String = "URLOP",
|
||||
val leaveRequestNote: String = "",
|
||||
val deletingPhotoIds: Set<String> = emptySet(),
|
||||
val completingRoute: Boolean = false,
|
||||
val imageAuthHeader: String? = null,
|
||||
@@ -62,11 +73,37 @@ data class DriverUiState(
|
||||
val isStale: Boolean = false,
|
||||
val lastSuccessfulSyncAtEpochMillis: Long? = null,
|
||||
val routeDayLiveUpdateMessage: String? = null,
|
||||
val themeMode: AppThemeMode = AppThemeMode.Default,
|
||||
val feedback: String? = null,
|
||||
val error: String? = null,
|
||||
)
|
||||
|
||||
private fun DriverUiState.withApiError(throwable: Throwable): DriverUiState {
|
||||
val apiError = ApiErrorMapper.map(throwable)
|
||||
return copy(
|
||||
isOnline = if (apiError.kind == ApiErrorKind.Network) false else isOnline,
|
||||
isStale = if (apiError.kind == ApiErrorKind.Network && lastSuccessfulSyncAtEpochMillis != null) true else isStale,
|
||||
error = apiError.message,
|
||||
)
|
||||
}
|
||||
|
||||
internal fun DriverUiState.withLoadedLeaveRequests(
|
||||
requests: List<DriverLeaveRequestDto>,
|
||||
navigateToList: Boolean,
|
||||
): DriverUiState {
|
||||
val refreshedSelectedRequest = selectedLeaveRequest?.let { selected ->
|
||||
requests.firstOrNull { request -> request.id == selected.id } ?: selected
|
||||
}
|
||||
return copy(
|
||||
screen = if (navigateToList) DriverScreen.LeaveRequests else screen,
|
||||
leaveRequests = requests,
|
||||
selectedLeaveRequest = refreshedSelectedRequest,
|
||||
error = null,
|
||||
)
|
||||
}
|
||||
|
||||
class DriverViewModel(application: Application) : AndroidViewModel(application) {
|
||||
private val appPreferencesStore = AppPreferencesStore(application)
|
||||
private val repository = DriverRepository(application)
|
||||
private val syncRepository = DriverSyncRepository(application, repository)
|
||||
private val networkMonitor = NetworkMonitor(application)
|
||||
@@ -85,6 +122,11 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
val state: StateFlow<DriverUiState> = _state
|
||||
|
||||
init {
|
||||
viewModelScope.launch {
|
||||
appPreferencesStore.themeMode.collect { themeMode ->
|
||||
_state.update { it.copy(themeMode = themeMode) }
|
||||
}
|
||||
}
|
||||
viewModelScope.launch {
|
||||
photoUploadOutbox.observeQueuedUploads().collect { uploads ->
|
||||
_state.update { it.copy(queuedPhotoUploads = uploads) }
|
||||
@@ -92,6 +134,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
viewModelScope.launch {
|
||||
networkMonitor.isOnline.collect { online ->
|
||||
liveSyncClient.onNetworkAvailable(online)
|
||||
_state.update {
|
||||
it.copy(
|
||||
isOnline = online,
|
||||
@@ -99,6 +142,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
)
|
||||
}
|
||||
if (online && repository.hasToken()) {
|
||||
liveSyncClient.ensureConnected()
|
||||
refreshCurrentScopeFromSyncState()
|
||||
}
|
||||
}
|
||||
@@ -112,7 +156,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
}
|
||||
|
||||
fun requestOtp(phone: String) = runLoading {
|
||||
fun requestOtp(phone: String) = runLoading("request_otp") {
|
||||
val response = repository.requestOtp(phone)
|
||||
otpAutoSubmitPolicy.reset()
|
||||
_state.update {
|
||||
@@ -126,8 +170,9 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
}
|
||||
|
||||
fun verifyOtp(code: String) = runLoading {
|
||||
fun verifyOtp(code: String) = runLoading("verify_otp") {
|
||||
val driver = repository.verifyOtp(_state.value.phone, code, android.os.Build.MODEL ?: "Android")
|
||||
AppDiagnostics.setDriverId(driver.id)
|
||||
_state.update { it.copy(driver = driver, imageAuthHeader = repository.imageAuthHeader()) }
|
||||
refreshRoutes()
|
||||
}
|
||||
@@ -145,6 +190,14 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
|
||||
fun refreshRoutesSilently() = loadRoutes(date = _state.value.selectedDate, showLoading = false, navigateToRoutes = false)
|
||||
|
||||
fun onAppForegrounded() {
|
||||
liveSyncClient.ensureConnected()
|
||||
viewModelScope.launch { checkRemoteSyncState() }
|
||||
if (_state.value.screen == DriverScreen.Routes) {
|
||||
refreshRoutesSilently()
|
||||
}
|
||||
}
|
||||
|
||||
fun selectRouteDate(date: String) {
|
||||
_state.update { it.copy(routeDayLiveUpdateMessage = null) }
|
||||
loadRoutes(date = date, showLoading = true, navigateToRoutes = true)
|
||||
@@ -162,9 +215,9 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
val cached = syncRepository.bootstrap(date)
|
||||
val response = cached.value
|
||||
val settings = response.driverAppSettings
|
||||
AppDiagnostics.setDriverId(response.session.driver.id)
|
||||
_state.update {
|
||||
it.copy(
|
||||
screen = if (navigateToRoutes) DriverScreen.Routes else it.screen,
|
||||
driver = response.session.driver,
|
||||
routes = response.routes.today,
|
||||
dispatchSheetReminder = response.dispatchSheetReminder,
|
||||
@@ -176,6 +229,13 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
allowRouteCompletion = settings?.allowRouteCompletion ?: it.allowRouteCompletion,
|
||||
requirePreciseLocationForPhotos = settings?.requirePreciseLocationForPhotos
|
||||
?: it.requirePreciseLocationForPhotos,
|
||||
leaveRequestsConfig = settings?.leaveRequests?.let { config ->
|
||||
LeaveRequestsConfig(enabled = config.enabled, types = config.types)
|
||||
},
|
||||
screen = if (
|
||||
settings?.leaveRequests?.enabled != true &&
|
||||
it.screen in setOf(DriverScreen.LeaveRequests, DriverScreen.LeaveRequestDetail, DriverScreen.AddLeaveRequest)
|
||||
) DriverScreen.Routes else if (navigateToRoutes) DriverScreen.Routes else it.screen,
|
||||
notifyNewRoutes = response.notificationPreferences?.notifyNewRoutes ?: it.notifyNewRoutes,
|
||||
imageAuthHeader = repository.imageAuthHeader(),
|
||||
isStale = cached.stale || !it.isOnline,
|
||||
@@ -189,16 +249,15 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
observeDispatchSheetUploads(response.dispatchSheetReminder?.workDate)
|
||||
}.onFailure { throwable ->
|
||||
reportHandledException("load_routes", throwable, mapOf("date" to date))
|
||||
_state.update {
|
||||
val apiError = ApiErrorMapper.map(throwable)
|
||||
it.copy(
|
||||
it.withApiError(throwable).copy(
|
||||
screen = when {
|
||||
apiError.kind == ApiErrorKind.Auth -> DriverScreen.Phone
|
||||
it.screen == DriverScreen.Initializing -> DriverScreen.Routes
|
||||
else -> it.screen
|
||||
},
|
||||
isStale = it.lastSuccessfulSyncAtEpochMillis != null,
|
||||
error = apiError.message,
|
||||
)
|
||||
}
|
||||
}
|
||||
@@ -209,7 +268,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
}
|
||||
|
||||
fun openRoute(routeId: String) = runLoading {
|
||||
fun openRoute(routeId: String) = runLoading("open_route", mapOf("route_id" to routeId)) {
|
||||
val cached = syncRepository.route(routeId)
|
||||
val response = cached.value
|
||||
photoUploadOutbox.discardConfirmedServerPhotos(response.route)
|
||||
@@ -247,7 +306,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
}
|
||||
.onFailure { throwable ->
|
||||
_state.update { it.copy(error = throwable.message ?: "Wystapil blad.") }
|
||||
_state.update { it.withApiError(throwable) }
|
||||
}
|
||||
|
||||
_state.update { it.copy(refreshing = false) }
|
||||
@@ -272,7 +331,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
}
|
||||
.onFailure { throwable ->
|
||||
_state.update { it.copy(error = ApiErrorMapper.map(throwable).message) }
|
||||
_state.update { it.withApiError(throwable) }
|
||||
}
|
||||
_state.update { it.copy(refreshing = false) }
|
||||
}
|
||||
@@ -288,17 +347,147 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
}
|
||||
|
||||
fun setThemeMode(themeMode: AppThemeMode) {
|
||||
_state.update { it.copy(themeMode = themeMode) }
|
||||
viewModelScope.launch {
|
||||
runCatching { appPreferencesStore.setThemeMode(themeMode) }
|
||||
.onFailure { throwable -> _state.update { it.withApiError(throwable) } }
|
||||
}
|
||||
}
|
||||
|
||||
fun openPhotoQueue() {
|
||||
_state.update { it.copy(screen = DriverScreen.PhotoQueue, error = null) }
|
||||
}
|
||||
|
||||
fun openLeaveRequests() {
|
||||
if (!DriverLeaveRequestUiRules.isFeatureVisible(_state.value.leaveRequestsConfig)) return
|
||||
loadLeaveRequests(showLoading = true, navigateToList = true)
|
||||
}
|
||||
|
||||
fun refreshLeaveRequests() {
|
||||
if (!DriverLeaveRequestUiRules.isFeatureVisible(_state.value.leaveRequestsConfig)) return
|
||||
loadLeaveRequests(showLoading = false, navigateToList = false)
|
||||
}
|
||||
|
||||
private fun loadLeaveRequests(showLoading: Boolean, navigateToList: Boolean) {
|
||||
viewModelScope.launch {
|
||||
_state.update { it.copy(loading = showLoading, refreshing = !showLoading, error = null, feedback = null) }
|
||||
runCatching { repository.leaveRequests() }
|
||||
.onSuccess { requests ->
|
||||
_state.update { it.withLoadedLeaveRequests(requests, navigateToList) }
|
||||
}
|
||||
.onFailure { throwable -> _state.update { it.withApiError(throwable) } }
|
||||
_state.update { it.copy(loading = false, refreshing = false) }
|
||||
}
|
||||
}
|
||||
|
||||
fun openLeaveRequest(id: String) {
|
||||
if (!DriverLeaveRequestUiRules.isFeatureVisible(_state.value.leaveRequestsConfig)) return
|
||||
viewModelScope.launch {
|
||||
_state.update { it.copy(loading = true, error = null, feedback = null) }
|
||||
runCatching { repository.leaveRequest(id) }
|
||||
.onSuccess { request ->
|
||||
_state.update { it.copy(screen = DriverScreen.LeaveRequestDetail, selectedLeaveRequest = request, error = null) }
|
||||
}
|
||||
.onFailure { throwable -> _state.update { it.withApiError(throwable) } }
|
||||
_state.update { it.copy(loading = false) }
|
||||
}
|
||||
}
|
||||
|
||||
fun openAddLeaveRequest() {
|
||||
val config = _state.value.leaveRequestsConfig
|
||||
if (!DriverLeaveRequestUiRules.isFeatureVisible(config)) return
|
||||
val today = LocalDate.now().toString()
|
||||
_state.update {
|
||||
it.copy(
|
||||
screen = DriverScreen.AddLeaveRequest,
|
||||
leaveRequestDateFrom = today,
|
||||
leaveRequestDateTo = today,
|
||||
leaveRequestType = config?.types?.firstOrNull() ?: "URLOP",
|
||||
leaveRequestNote = "",
|
||||
error = null,
|
||||
feedback = null,
|
||||
)
|
||||
}
|
||||
}
|
||||
|
||||
fun updateLeaveRequestDraft(dateFrom: String? = null, dateTo: String? = null, type: String? = null, note: String? = null) {
|
||||
_state.update {
|
||||
val nextFrom = dateFrom ?: it.leaveRequestDateFrom
|
||||
val nextTo = dateTo ?: it.leaveRequestDateTo
|
||||
it.copy(
|
||||
leaveRequestDateFrom = nextFrom,
|
||||
leaveRequestDateTo = if (nextTo < nextFrom) nextFrom else nextTo,
|
||||
leaveRequestType = type ?: it.leaveRequestType,
|
||||
leaveRequestNote = note ?: it.leaveRequestNote,
|
||||
)
|
||||
}
|
||||
}
|
||||
|
||||
fun submitLeaveRequest() {
|
||||
val snapshot = _state.value
|
||||
if (!DriverLeaveRequestUiRules.isFeatureVisible(snapshot.leaveRequestsConfig)) return
|
||||
if (snapshot.leaveRequestDateFrom < LocalDate.now().toString()) {
|
||||
_state.update { it.copy(error = "Data od nie może być z przeszłości.") }
|
||||
return
|
||||
}
|
||||
|
||||
viewModelScope.launch {
|
||||
_state.update { it.copy(loading = true, error = null, feedback = null) }
|
||||
runCatching {
|
||||
repository.createLeaveRequest(
|
||||
dateFrom = snapshot.leaveRequestDateFrom,
|
||||
dateTo = snapshot.leaveRequestDateTo,
|
||||
type = snapshot.leaveRequestType,
|
||||
note = snapshot.leaveRequestNote.takeIf { it.isNotBlank() },
|
||||
)
|
||||
}.onSuccess { created ->
|
||||
val requests = runCatching { repository.leaveRequests() }.getOrElse { listOf(created) }
|
||||
_state.update {
|
||||
it.copy(
|
||||
screen = DriverScreen.LeaveRequestDetail,
|
||||
selectedLeaveRequest = created,
|
||||
leaveRequests = requests,
|
||||
feedback = "Wniosek został wysłany do decyzji.",
|
||||
error = null,
|
||||
)
|
||||
}
|
||||
}.onFailure { throwable ->
|
||||
_state.update { it.withApiError(throwable) }
|
||||
}
|
||||
_state.update { it.copy(loading = false) }
|
||||
}
|
||||
}
|
||||
|
||||
fun cancelSelectedLeaveRequest() {
|
||||
val request = _state.value.selectedLeaveRequest ?: return
|
||||
if (!DriverLeaveRequestUiRules.canCancel(request.status, request.dateFrom.orEmpty())) return
|
||||
|
||||
viewModelScope.launch {
|
||||
_state.update { it.copy(loading = true, error = null, feedback = null) }
|
||||
runCatching { repository.cancelLeaveRequest(request.id) }
|
||||
.onSuccess { updated ->
|
||||
_state.update {
|
||||
it.copy(
|
||||
selectedLeaveRequest = updated,
|
||||
leaveRequests = it.leaveRequests.map { existing -> if (existing.id == updated.id) updated else existing },
|
||||
feedback = if (updated.status == "cancel_requested") "Anulowanie wysłane do decyzji." else "Wniosek został anulowany.",
|
||||
error = null,
|
||||
)
|
||||
}
|
||||
}
|
||||
.onFailure { throwable -> _state.update { it.withApiError(throwable) } }
|
||||
_state.update { it.copy(loading = false) }
|
||||
}
|
||||
}
|
||||
|
||||
fun uploadPhoto(uri: Uri, source: String, metadata: PhotoUploadMetadata = PhotoUploadMetadata()) {
|
||||
val route = _state.value.selectedRoute ?: return
|
||||
viewModelScope.launch {
|
||||
_state.update { it.copy(error = null) }
|
||||
runCatching { photoUploadOutbox.enqueue(route.id, uri, source, metadata) }
|
||||
.onFailure { throwable ->
|
||||
_state.update { it.copy(error = ApiErrorMapper.map(throwable).message) }
|
||||
_state.update { it.withApiError(throwable) }
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -319,7 +508,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
metadata = metadata,
|
||||
)
|
||||
}.onFailure { throwable ->
|
||||
_state.update { it.copy(error = ApiErrorMapper.map(throwable).message) }
|
||||
_state.update { it.withApiError(throwable) }
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -328,7 +517,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
viewModelScope.launch {
|
||||
runCatching { photoUploadOutbox.retry(upload.clientRequestId) }
|
||||
.onFailure { throwable ->
|
||||
_state.update { it.copy(error = ApiErrorMapper.map(throwable).message) }
|
||||
_state.update { it.withApiError(throwable) }
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -349,7 +538,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
)
|
||||
}
|
||||
}.onFailure { throwable ->
|
||||
_state.update { it.copy(error = ApiErrorMapper.map(throwable).message) }
|
||||
_state.update { it.withApiError(throwable) }
|
||||
}
|
||||
_state.update { it.copy(deletingPhotoIds = it.deletingPhotoIds - photo.id) }
|
||||
}
|
||||
@@ -374,7 +563,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
)
|
||||
}
|
||||
}.onFailure { throwable ->
|
||||
_state.update { it.copy(error = ApiErrorMapper.map(throwable).message) }
|
||||
_state.update { it.withApiError(throwable) }
|
||||
}
|
||||
_state.update { it.copy(deletingPhotoIds = it.deletingPhotoIds - serverPhotoId) }
|
||||
}
|
||||
@@ -405,7 +594,7 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
)
|
||||
}
|
||||
}.onFailure { throwable ->
|
||||
_state.update { it.copy(feedback = null, error = ApiErrorMapper.map(throwable).message) }
|
||||
_state.update { it.withApiError(throwable).copy(feedback = null) }
|
||||
}
|
||||
_state.update { it.copy(completingRoute = false) }
|
||||
}
|
||||
@@ -425,6 +614,9 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
when (it.screen) {
|
||||
DriverScreen.Photo -> it.copy(screen = DriverScreen.Detail, selectedPhoto = null)
|
||||
DriverScreen.PhotoQueue -> it.copy(screen = DriverScreen.Routes)
|
||||
DriverScreen.LeaveRequests -> it.copy(screen = DriverScreen.Routes)
|
||||
DriverScreen.LeaveRequestDetail -> it.copy(screen = DriverScreen.LeaveRequests, selectedLeaveRequest = null, feedback = null)
|
||||
DriverScreen.AddLeaveRequest -> it.copy(screen = DriverScreen.LeaveRequests, feedback = null)
|
||||
DriverScreen.Detail -> {
|
||||
photoUploadsJob?.cancel()
|
||||
it.copy(screen = DriverScreen.Routes, selectedRoute = null, photoUploads = emptyList(), feedback = null)
|
||||
@@ -436,22 +628,28 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
}
|
||||
|
||||
fun logout() = runLoading {
|
||||
fun logout() = runLoading("logout") {
|
||||
photoUploadsJob?.cancel()
|
||||
dispatchSheetUploadsJob?.cancel()
|
||||
liveSyncClient.stop()
|
||||
repository.logout()
|
||||
syncRepository.clearCache()
|
||||
pushTokenRegisteredForDriverId = null
|
||||
AppDiagnostics.clearDriverId()
|
||||
_state.update { DriverUiState(screen = DriverScreen.Phone, loading = false) }
|
||||
}
|
||||
|
||||
private fun runLoading(block: suspend () -> Unit) {
|
||||
private fun runLoading(
|
||||
operation: String,
|
||||
keys: Map<String, Any?> = emptyMap(),
|
||||
block: suspend () -> Unit,
|
||||
) {
|
||||
viewModelScope.launch {
|
||||
_state.update { it.copy(loading = true, error = null) }
|
||||
runCatching { block() }
|
||||
.onFailure { throwable ->
|
||||
_state.update { it.copy(error = ApiErrorMapper.map(throwable).message) }
|
||||
reportHandledException(operation, throwable, keys)
|
||||
_state.update { it.withApiError(throwable) }
|
||||
}
|
||||
_state.update { it.copy(loading = false) }
|
||||
}
|
||||
@@ -489,6 +687,9 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
if (snapshot.screen == DriverScreen.Detail) {
|
||||
refreshSelectedRoute()
|
||||
}
|
||||
if (snapshot.screen == DriverScreen.LeaveRequests || snapshot.screen == DriverScreen.LeaveRequestDetail) {
|
||||
refreshLeaveRequests()
|
||||
}
|
||||
}
|
||||
|
||||
private suspend fun checkRemoteSyncState() {
|
||||
@@ -530,6 +731,11 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
}
|
||||
DriverSyncRepository.SCOPE_SETTINGS -> refreshRoutesSilently()
|
||||
DriverSyncRepository.SCOPE_DISPATCH_SHEET -> refreshRoutesSilently()
|
||||
DriverSyncRepository.SCOPE_LEAVE_REQUESTS -> {
|
||||
if (snapshot.screen == DriverScreen.LeaveRequests || snapshot.screen == DriverScreen.LeaveRequestDetail) {
|
||||
refreshLeaveRequests()
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -542,10 +748,39 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
|
||||
runCatching {
|
||||
repository.storePushToken(token)
|
||||
pushTokenRegisteredForDriverId = driverId
|
||||
}.onFailure { throwable ->
|
||||
reportHandledException("store_push_token", throwable, mapOf("driver_id" to driverId))
|
||||
}
|
||||
}
|
||||
}.addOnFailureListener { throwable ->
|
||||
reportHandledException("get_push_token", throwable, mapOf("driver_id" to driverId))
|
||||
}
|
||||
}.onFailure { throwable ->
|
||||
reportHandledException("register_push_token", throwable, mapOf("driver_id" to driverId))
|
||||
}
|
||||
}
|
||||
|
||||
private fun reportHandledException(
|
||||
operation: String,
|
||||
throwable: Throwable,
|
||||
keys: Map<String, Any?> = emptyMap(),
|
||||
) {
|
||||
val apiError = ApiErrorMapper.map(throwable)
|
||||
if (!ApiErrorMapper.shouldReportNonFatal(throwable)) {
|
||||
AppDiagnostics.log("network_failure: $operation: ${throwable.message ?: throwable::class.java.simpleName}")
|
||||
return
|
||||
}
|
||||
AppDiagnostics.reportNonFatal(
|
||||
throwable = throwable,
|
||||
operation = operation,
|
||||
keys = mapOf(
|
||||
"screen" to _state.value.screen.name,
|
||||
"api_error_kind" to apiError.kind.name,
|
||||
"api_error_code" to apiError.code,
|
||||
"api_status_code" to apiError.statusCode,
|
||||
"retryable" to apiError.retryable,
|
||||
) + keys,
|
||||
)
|
||||
}
|
||||
|
||||
override fun onCleared() {
|
||||
|
||||
@@ -0,0 +1,13 @@
|
||||
package pl.firmatpp.kierowca.ui.theme
|
||||
|
||||
enum class AppThemeMode(val storedValue: String) {
|
||||
Current("current"),
|
||||
Material3("material3");
|
||||
|
||||
companion object {
|
||||
val Default: AppThemeMode = Material3
|
||||
|
||||
fun fromStoredValue(value: String?): AppThemeMode =
|
||||
entries.firstOrNull { it.storedValue == value } ?: Default
|
||||
}
|
||||
}
|
||||
@@ -1,9 +1,20 @@
|
||||
package pl.firmatpp.kierowca.ui.theme
|
||||
|
||||
import android.app.Activity
|
||||
import androidx.compose.material3.MaterialTheme
|
||||
import androidx.compose.material3.Shapes
|
||||
import androidx.compose.material3.lightColorScheme
|
||||
import androidx.compose.runtime.Composable
|
||||
import androidx.compose.runtime.CompositionLocalProvider
|
||||
import androidx.compose.runtime.Immutable
|
||||
import androidx.compose.runtime.ReadOnlyComposable
|
||||
import androidx.compose.runtime.SideEffect
|
||||
import androidx.compose.runtime.staticCompositionLocalOf
|
||||
import androidx.compose.ui.graphics.Color
|
||||
import androidx.compose.ui.graphics.toArgb
|
||||
import androidx.compose.ui.platform.LocalView
|
||||
import androidx.compose.ui.unit.dp
|
||||
import androidx.core.view.WindowCompat
|
||||
|
||||
object TppColors {
|
||||
val Forest = Color(0xFF154212)
|
||||
@@ -18,7 +29,71 @@ object TppColors {
|
||||
val Error = Color(0xFFBA1A1A)
|
||||
}
|
||||
|
||||
private val TppScheme = lightColorScheme(
|
||||
@Immutable
|
||||
data class TppColorTokens(
|
||||
val forest: Color,
|
||||
val containerGreen: Color,
|
||||
val logoLeaf: Color,
|
||||
val navy: Color,
|
||||
val surface: Color,
|
||||
val panel: Color,
|
||||
val ink: Color,
|
||||
val muted: Color,
|
||||
val outline: Color,
|
||||
val error: Color,
|
||||
val card: Color,
|
||||
val successContainer: Color,
|
||||
val successOutline: Color,
|
||||
val warningContainer: Color,
|
||||
val warningOutline: Color,
|
||||
)
|
||||
|
||||
object TppTheme {
|
||||
val colors: TppColorTokens
|
||||
@Composable
|
||||
@ReadOnlyComposable
|
||||
get() = LocalTppColorTokens.current
|
||||
}
|
||||
|
||||
private val CurrentColorTokens = TppColorTokens(
|
||||
forest = TppColors.Forest,
|
||||
containerGreen = TppColors.ContainerGreen,
|
||||
logoLeaf = TppColors.LogoLeaf,
|
||||
navy = TppColors.Navy,
|
||||
surface = TppColors.Surface,
|
||||
panel = TppColors.Panel,
|
||||
ink = TppColors.Ink,
|
||||
muted = TppColors.Muted,
|
||||
outline = TppColors.Outline,
|
||||
error = TppColors.Error,
|
||||
card = Color.White,
|
||||
successContainer = Color(0xFFEAF7EF),
|
||||
successOutline = Color(0xFF9BD1AD),
|
||||
warningContainer = Color(0xFFFFF7E6),
|
||||
warningOutline = Color(0xFFE6B85C),
|
||||
)
|
||||
|
||||
private val Material3ColorTokens = TppColorTokens(
|
||||
forest = Color(0xFF226C2C),
|
||||
containerGreen = Color(0xFF2F7D32),
|
||||
logoLeaf = Color(0xFF5FA642),
|
||||
navy = Color(0xFF225A86),
|
||||
surface = Color(0xFFF8FBF6),
|
||||
panel = Color(0xFFEAF1E6),
|
||||
ink = Color(0xFF172018),
|
||||
muted = Color(0xFF566252),
|
||||
outline = Color(0xFFBAC8B5),
|
||||
error = Color(0xFFBA1A1A),
|
||||
card = Color(0xFFFFFFFF),
|
||||
successContainer = Color(0xFFDFF4E2),
|
||||
successOutline = Color(0xFF8EC69A),
|
||||
warningContainer = Color(0xFFFFF1D6),
|
||||
warningOutline = Color(0xFFE4B45B),
|
||||
)
|
||||
|
||||
private val LocalTppColorTokens = staticCompositionLocalOf { CurrentColorTokens }
|
||||
|
||||
private val CurrentScheme = lightColorScheme(
|
||||
primary = TppColors.Forest,
|
||||
onPrimary = Color.White,
|
||||
primaryContainer = TppColors.ContainerGreen,
|
||||
@@ -33,10 +108,71 @@ private val TppScheme = lightColorScheme(
|
||||
error = TppColors.Error,
|
||||
)
|
||||
|
||||
private val Material3Scheme = lightColorScheme(
|
||||
primary = Material3ColorTokens.forest,
|
||||
onPrimary = Color.White,
|
||||
primaryContainer = Material3ColorTokens.containerGreen,
|
||||
onPrimaryContainer = Color.White,
|
||||
secondary = Material3ColorTokens.navy,
|
||||
onSecondary = Color.White,
|
||||
secondaryContainer = Color(0xFFD2E7F7),
|
||||
onSecondaryContainer = Color(0xFF071E30),
|
||||
tertiary = Color(0xFF7A5D00),
|
||||
onTertiary = Color.White,
|
||||
tertiaryContainer = Color(0xFFFFE08A),
|
||||
background = Material3ColorTokens.surface,
|
||||
onBackground = Material3ColorTokens.ink,
|
||||
surface = Material3ColorTokens.surface,
|
||||
onSurface = Material3ColorTokens.ink,
|
||||
surfaceVariant = Material3ColorTokens.panel,
|
||||
onSurfaceVariant = Material3ColorTokens.muted,
|
||||
outline = Material3ColorTokens.outline,
|
||||
outlineVariant = Color(0xFFD5DED1),
|
||||
error = Material3ColorTokens.error,
|
||||
)
|
||||
|
||||
private val CurrentShapes = Shapes(
|
||||
extraSmall = androidx.compose.foundation.shape.RoundedCornerShape(2.dp),
|
||||
small = androidx.compose.foundation.shape.RoundedCornerShape(4.dp),
|
||||
medium = androidx.compose.foundation.shape.RoundedCornerShape(8.dp),
|
||||
large = androidx.compose.foundation.shape.RoundedCornerShape(8.dp),
|
||||
extraLarge = androidx.compose.foundation.shape.RoundedCornerShape(12.dp),
|
||||
)
|
||||
|
||||
private val Material3Shapes = Shapes(
|
||||
extraSmall = androidx.compose.foundation.shape.RoundedCornerShape(8.dp),
|
||||
small = androidx.compose.foundation.shape.RoundedCornerShape(12.dp),
|
||||
medium = androidx.compose.foundation.shape.RoundedCornerShape(20.dp),
|
||||
large = androidx.compose.foundation.shape.RoundedCornerShape(28.dp),
|
||||
extraLarge = androidx.compose.foundation.shape.RoundedCornerShape(32.dp),
|
||||
)
|
||||
|
||||
@Composable
|
||||
fun TppKierowcaTheme(content: @Composable () -> Unit) {
|
||||
@Suppress("DEPRECATION")
|
||||
fun TppKierowcaTheme(
|
||||
themeMode: AppThemeMode = AppThemeMode.Default,
|
||||
content: @Composable () -> Unit,
|
||||
) {
|
||||
val colorTokens = if (themeMode == AppThemeMode.Material3) Material3ColorTokens else CurrentColorTokens
|
||||
val colorScheme = if (themeMode == AppThemeMode.Material3) Material3Scheme else CurrentScheme
|
||||
val shapes = if (themeMode == AppThemeMode.Material3) Material3Shapes else CurrentShapes
|
||||
val view = LocalView.current
|
||||
|
||||
if (!view.isInEditMode) {
|
||||
SideEffect {
|
||||
val window = (view.context as? Activity)?.window ?: return@SideEffect
|
||||
window.statusBarColor = colorTokens.surface.toArgb()
|
||||
window.navigationBarColor = colorTokens.surface.toArgb()
|
||||
WindowCompat.getInsetsController(window, view).isAppearanceLightStatusBars = true
|
||||
WindowCompat.getInsetsController(window, view).isAppearanceLightNavigationBars = true
|
||||
}
|
||||
}
|
||||
|
||||
CompositionLocalProvider(LocalTppColorTokens provides colorTokens) {
|
||||
MaterialTheme(
|
||||
colorScheme = TppScheme,
|
||||
colorScheme = colorScheme,
|
||||
shapes = shapes,
|
||||
content = content,
|
||||
)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -0,0 +1,4 @@
|
||||
<?xml version="1.0" encoding="utf-8"?>
|
||||
<resources>
|
||||
<style name="Theme.ITSTransport" parent="android:Theme.Material.Light.NoActionBar" />
|
||||
</resources>
|
||||
@@ -1,6 +1,7 @@
|
||||
package pl.firmatpp.kierowca.data
|
||||
|
||||
import java.io.IOException
|
||||
import java.net.UnknownHostException
|
||||
import org.junit.Assert.assertEquals
|
||||
import org.junit.Assert.assertFalse
|
||||
import org.junit.Assert.assertTrue
|
||||
@@ -16,6 +17,23 @@ class ApiErrorMapperTest {
|
||||
assertEquals("Nie udało się połączyć z serwerem. Operacja nie została potwierdzona.", error.message)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun mapsUnknownHostToActionableOfflineMessage() {
|
||||
val error = ApiErrorMapper.map(UnknownHostException("api-intranet.firmatpp.pl"))
|
||||
|
||||
assertEquals(ApiErrorKind.Network, error.kind)
|
||||
assertTrue(error.retryable)
|
||||
assertEquals(
|
||||
"Brak internetu lub połączenia z serwerem. Sprawdź zasięg i spróbuj ponownie.",
|
||||
error.message,
|
||||
)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun doesNotReportExpectedNetworkFailuresAsNonFatal() {
|
||||
assertFalse(ApiErrorMapper.shouldReportNonFatal(UnknownHostException("api-intranet.firmatpp.pl")))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun mapsMobileApiProblemCodes() {
|
||||
val error = ApiErrorMapper.mapProblem(
|
||||
|
||||
@@ -0,0 +1,44 @@
|
||||
package pl.firmatpp.kierowca.data
|
||||
|
||||
import androidx.datastore.preferences.core.PreferenceDataStoreFactory
|
||||
import java.io.File
|
||||
import kotlinx.coroutines.ExperimentalCoroutinesApi
|
||||
import kotlinx.coroutines.flow.first
|
||||
import kotlinx.coroutines.test.TestScope
|
||||
import kotlinx.coroutines.test.runTest
|
||||
import org.junit.Assert.assertEquals
|
||||
import org.junit.Rule
|
||||
import org.junit.Test
|
||||
import org.junit.rules.TemporaryFolder
|
||||
import pl.firmatpp.kierowca.ui.theme.AppThemeMode
|
||||
|
||||
@OptIn(ExperimentalCoroutinesApi::class)
|
||||
class AppPreferencesStoreTest {
|
||||
@get:Rule
|
||||
val temporaryFolder = TemporaryFolder()
|
||||
|
||||
@Test
|
||||
fun startsWithMaterial3ThemeMode() = runTest {
|
||||
val store = preferencesStore("default.preferences_pb")
|
||||
|
||||
assertEquals(AppThemeMode.Material3, store.themeMode.first())
|
||||
}
|
||||
|
||||
@Test
|
||||
fun persistsSelectedThemeMode() = runTest {
|
||||
val file = temporaryFolder.newFile("theme.preferences_pb")
|
||||
val store = preferencesStore(file)
|
||||
|
||||
store.setThemeMode(AppThemeMode.Material3)
|
||||
|
||||
assertEquals(AppThemeMode.Material3, store.themeMode.first())
|
||||
}
|
||||
|
||||
private fun TestScope.preferencesStore(fileName: String): AppPreferencesStore =
|
||||
preferencesStore(temporaryFolder.newFile(fileName))
|
||||
|
||||
private fun TestScope.preferencesStore(file: File): AppPreferencesStore {
|
||||
val dataStore = PreferenceDataStoreFactory.create(scope = backgroundScope) { file }
|
||||
return AppPreferencesStore(dataStore)
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,71 @@
|
||||
package pl.firmatpp.kierowca.diagnostics
|
||||
|
||||
import org.junit.After
|
||||
import org.junit.Assert.assertEquals
|
||||
import org.junit.Assert.assertSame
|
||||
import org.junit.Test
|
||||
|
||||
class AppDiagnosticsTest {
|
||||
private val sink = RecordingDiagnosticsSink()
|
||||
|
||||
@After
|
||||
fun tearDown() {
|
||||
AppDiagnostics.installSink(NoOpDiagnosticsSink)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun reportNonFatalAddsOperationContextBeforeRecordingException() {
|
||||
AppDiagnostics.installSink(sink)
|
||||
val throwable = IllegalStateException("upload failed")
|
||||
|
||||
AppDiagnostics.reportNonFatal(
|
||||
throwable = throwable,
|
||||
operation = "photo_upload",
|
||||
keys = mapOf("route_id" to "R-42", "retryable" to true),
|
||||
)
|
||||
|
||||
assertEquals(
|
||||
listOf(
|
||||
"key:operation=photo_upload",
|
||||
"key:route_id=R-42",
|
||||
"key:retryable=true",
|
||||
"log:non_fatal: photo_upload: upload failed",
|
||||
"exception:IllegalStateException",
|
||||
),
|
||||
sink.events,
|
||||
)
|
||||
assertSame(throwable, sink.throwables.single())
|
||||
}
|
||||
|
||||
@Test
|
||||
fun clearDriverIdClearsPreviouslySetUserIdentifier() {
|
||||
AppDiagnostics.installSink(sink)
|
||||
|
||||
AppDiagnostics.setDriverId("driver-7")
|
||||
AppDiagnostics.clearDriverId()
|
||||
|
||||
assertEquals(listOf("user:driver-7", "user:"), sink.events)
|
||||
}
|
||||
|
||||
private class RecordingDiagnosticsSink : DiagnosticsSink {
|
||||
val events = mutableListOf<String>()
|
||||
val throwables = mutableListOf<Throwable>()
|
||||
|
||||
override fun log(message: String) {
|
||||
events += "log:$message"
|
||||
}
|
||||
|
||||
override fun setUserId(userId: String) {
|
||||
events += "user:$userId"
|
||||
}
|
||||
|
||||
override fun setCustomKey(key: String, value: String) {
|
||||
events += "key:$key=$value"
|
||||
}
|
||||
|
||||
override fun recordException(throwable: Throwable) {
|
||||
events += "exception:${throwable::class.simpleName}"
|
||||
throwables += throwable
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,217 @@
|
||||
package pl.firmatpp.kierowca.sync
|
||||
|
||||
import kotlinx.coroutines.CoroutineScope
|
||||
import kotlinx.coroutines.ExperimentalCoroutinesApi
|
||||
import kotlinx.coroutines.test.advanceTimeBy
|
||||
import kotlinx.coroutines.test.advanceUntilIdle
|
||||
import kotlinx.coroutines.test.runTest
|
||||
import kotlinx.coroutines.test.runCurrent
|
||||
import okhttp3.Request
|
||||
import okhttp3.Response
|
||||
import okhttp3.WebSocket
|
||||
import okhttp3.WebSocketListener
|
||||
import okio.ByteString
|
||||
import org.junit.Assert.assertEquals
|
||||
import org.junit.Assert.assertTrue
|
||||
import org.junit.Test
|
||||
import pl.firmatpp.kierowca.data.model.BroadcastAuthResponse
|
||||
import pl.firmatpp.kierowca.data.model.RealtimeConfigDto
|
||||
|
||||
@OptIn(ExperimentalCoroutinesApi::class)
|
||||
class DriverLiveSyncClientTest {
|
||||
private val config = RealtimeConfigDto(
|
||||
reverbEnabled = true,
|
||||
reverbAppKey = "app-key",
|
||||
reverbWsBaseUrl = "wss://example.test/app",
|
||||
)
|
||||
|
||||
@Test
|
||||
fun reconnectsWithBackoffAfterSocketFailure() = runTest {
|
||||
val factory = FakeWebSocketFactory()
|
||||
val client = liveClient(factory = factory, scope = backgroundScope)
|
||||
|
||||
client.start("driver-1", config)
|
||||
factory.sockets.single().fail()
|
||||
|
||||
advanceTimeBy(999)
|
||||
assertEquals(1, factory.sockets.size)
|
||||
|
||||
advanceTimeBy(1)
|
||||
runCurrent()
|
||||
assertEquals(2, factory.sockets.size)
|
||||
|
||||
factory.sockets.last().fail()
|
||||
advanceTimeBy(1_999)
|
||||
assertEquals(2, factory.sockets.size)
|
||||
|
||||
advanceTimeBy(1)
|
||||
runCurrent()
|
||||
assertEquals(3, factory.sockets.size)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun pausesReconnectWhileOfflineAndReconnectsImmediatelyWhenNetworkReturns() = runTest {
|
||||
val factory = FakeWebSocketFactory()
|
||||
val client = liveClient(factory = factory, scope = backgroundScope)
|
||||
|
||||
client.start("driver-1", config)
|
||||
client.onNetworkAvailable(false)
|
||||
factory.sockets.single().fail()
|
||||
|
||||
advanceTimeBy(10_000)
|
||||
assertEquals(1, factory.sockets.size)
|
||||
assertTrue(factory.sockets.single().closed)
|
||||
|
||||
client.onNetworkAvailable(true)
|
||||
advanceUntilIdle()
|
||||
|
||||
assertEquals(2, factory.sockets.size)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun foregroundEnsureConnectedRestartsWhenSocketIsDisconnected() = runTest {
|
||||
val factory = FakeWebSocketFactory()
|
||||
val client = liveClient(factory = factory, scope = backgroundScope)
|
||||
|
||||
client.start("driver-1", config)
|
||||
factory.sockets.single().fail()
|
||||
client.ensureConnected()
|
||||
advanceUntilIdle()
|
||||
|
||||
assertEquals(2, factory.sockets.size)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun retriesWhenPrivateChannelSubscriptionAuthFails() = runTest {
|
||||
val gateway = FakeLiveSyncGateway(authFailure = RuntimeException("offline"))
|
||||
val factory = FakeWebSocketFactory()
|
||||
val client = liveClient(gateway = gateway, factory = factory, scope = backgroundScope)
|
||||
|
||||
client.start("driver-1", config)
|
||||
factory.sockets.single().message(connectionEstablished("socket-1"))
|
||||
advanceUntilIdle()
|
||||
advanceTimeBy(1_000)
|
||||
runCurrent()
|
||||
|
||||
assertEquals(2, factory.sockets.size)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun respondsToPusherPingWithPong() = runTest {
|
||||
val factory = FakeWebSocketFactory()
|
||||
val client = liveClient(factory = factory, scope = backgroundScope)
|
||||
|
||||
client.start("driver-1", config)
|
||||
factory.sockets.single().message("""{"event":"pusher:ping"}""")
|
||||
|
||||
assertTrue(factory.sockets.single().sent.any { it.contains("pusher:pong") })
|
||||
}
|
||||
|
||||
@Test
|
||||
fun closesAndReconnectsWhenConnectedSocketStopsReceivingMessages() = runTest {
|
||||
val factory = FakeWebSocketFactory()
|
||||
val client = liveClient(factory = factory, scope = backgroundScope)
|
||||
|
||||
client.start("driver-1", config)
|
||||
factory.sockets.single().message(connectionEstablished("socket-1"))
|
||||
advanceUntilIdle()
|
||||
factory.sockets.single().message("""{"event":"pusher_internal:subscription_succeeded","channel":"private-driver-mobile.driver-1"}""")
|
||||
|
||||
advanceTimeBy(30_000)
|
||||
runCurrent()
|
||||
|
||||
assertTrue(factory.sockets.first().closed)
|
||||
assertEquals(2, factory.sockets.size)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun ignoresCallbacksFromOldSocketAfterReconnect() = runTest {
|
||||
val factory = FakeWebSocketFactory()
|
||||
val client = liveClient(factory = factory, scope = backgroundScope)
|
||||
|
||||
client.start("driver-1", config)
|
||||
val oldSocket = factory.sockets.single()
|
||||
oldSocket.fail()
|
||||
client.ensureConnected()
|
||||
assertEquals(2, factory.sockets.size)
|
||||
|
||||
oldSocket.fail()
|
||||
client.ensureConnected()
|
||||
|
||||
assertEquals(2, factory.sockets.size)
|
||||
}
|
||||
|
||||
private fun liveClient(
|
||||
gateway: FakeLiveSyncGateway = FakeLiveSyncGateway(),
|
||||
factory: FakeWebSocketFactory,
|
||||
scope: CoroutineScope,
|
||||
): DriverLiveSyncClient =
|
||||
DriverLiveSyncClient(
|
||||
gateway = gateway,
|
||||
onConnected = {},
|
||||
onHint = {},
|
||||
webSocketFactory = factory,
|
||||
scope = scope,
|
||||
reconnectDelaysMs = listOf(1_000L, 2_000L),
|
||||
staleTimeoutMs = 30_000L,
|
||||
)
|
||||
|
||||
private fun connectionEstablished(socketId: String): String =
|
||||
"""{"event":"pusher:connection_established","data":"{\"socket_id\":\"$socketId\"}"}"""
|
||||
|
||||
private class FakeLiveSyncGateway(
|
||||
private val authFailure: Throwable? = null,
|
||||
) : DriverLiveSyncGateway {
|
||||
override suspend fun broadcastAuth(socketId: String, channelName: String): BroadcastAuthResponse {
|
||||
authFailure?.let { throw it }
|
||||
return BroadcastAuthResponse(auth = "auth-token")
|
||||
}
|
||||
|
||||
override suspend fun storeRealtimeStatus(status: String, socketId: String?, error: String?) = Unit
|
||||
}
|
||||
|
||||
private class FakeWebSocketFactory : LiveWebSocketFactory {
|
||||
val sockets = mutableListOf<FakeWebSocket>()
|
||||
|
||||
override fun newWebSocket(url: String, listener: WebSocketListener): WebSocket {
|
||||
val socket = FakeWebSocket(listener)
|
||||
sockets += socket
|
||||
return socket
|
||||
}
|
||||
}
|
||||
|
||||
private class FakeWebSocket(
|
||||
private val listener: WebSocketListener,
|
||||
) : WebSocket {
|
||||
val sent = mutableListOf<String>()
|
||||
var closed = false
|
||||
|
||||
fun fail() {
|
||||
listener.onFailure(this, RuntimeException("boom"), null)
|
||||
}
|
||||
|
||||
fun message(text: String) {
|
||||
listener.onMessage(this, text)
|
||||
}
|
||||
|
||||
override fun request(): Request = Request.Builder().url("wss://example.test/app").build()
|
||||
|
||||
override fun queueSize(): Long = 0L
|
||||
|
||||
override fun send(text: String): Boolean {
|
||||
sent += text
|
||||
return true
|
||||
}
|
||||
|
||||
override fun send(bytes: ByteString): Boolean = true
|
||||
|
||||
override fun close(code: Int, reason: String?): Boolean {
|
||||
closed = true
|
||||
return true
|
||||
}
|
||||
|
||||
override fun cancel() {
|
||||
closed = true
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,51 @@
|
||||
package pl.firmatpp.kierowca.ui
|
||||
|
||||
import java.time.LocalDate
|
||||
import org.junit.Assert.assertEquals
|
||||
import org.junit.Assert.assertFalse
|
||||
import org.junit.Assert.assertTrue
|
||||
import org.junit.Test
|
||||
|
||||
class DriverLeaveRequestUiRulesTest {
|
||||
@Test
|
||||
fun featureIsVisibleOnlyWhenServerConfigEnablesIt() {
|
||||
assertFalse(DriverLeaveRequestUiRules.isFeatureVisible(null))
|
||||
assertFalse(DriverLeaveRequestUiRules.isFeatureVisible(LeaveRequestsConfig(enabled = false, types = listOf("URLOP"))))
|
||||
assertTrue(DriverLeaveRequestUiRules.isFeatureVisible(LeaveRequestsConfig(enabled = true, types = listOf("URLOP"))))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun exposesPolishStatusLabels() {
|
||||
assertEquals("Oczekuje", DriverLeaveRequestUiRules.statusLabel("pending"))
|
||||
assertEquals("Zatwierdzony", DriverLeaveRequestUiRules.statusLabel("approved"))
|
||||
assertEquals("Odrzucony", DriverLeaveRequestUiRules.statusLabel("rejected"))
|
||||
assertEquals("Anulowanie do decyzji", DriverLeaveRequestUiRules.statusLabel("cancel_requested"))
|
||||
assertEquals("Anulowany", DriverLeaveRequestUiRules.statusLabel("cancelled"))
|
||||
assertEquals("Cofnięto decyzję", DriverLeaveRequestUiRules.statusLabel("revoked"))
|
||||
assertEquals("Nieznany status", DriverLeaveRequestUiRules.statusLabel("some_raw_status"))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun exposesPolishEventActionLabels() {
|
||||
assertEquals("Wniosek złożony", DriverLeaveRequestUiRules.eventActionLabel("submitted"))
|
||||
assertEquals("Wniosek zatwierdzony", DriverLeaveRequestUiRules.eventActionLabel("approved"))
|
||||
assertEquals("Wniosek odrzucony", DriverLeaveRequestUiRules.eventActionLabel("rejected"))
|
||||
assertEquals("Decyzja cofnięta", DriverLeaveRequestUiRules.eventActionLabel("revoked"))
|
||||
assertEquals("Wniosek anulowany przez kierowcę", DriverLeaveRequestUiRules.eventActionLabel("cancelled_by_driver"))
|
||||
assertEquals("Kierowca poprosił o anulowanie", DriverLeaveRequestUiRules.eventActionLabel("cancellation_requested"))
|
||||
assertEquals("Anulowanie zatwierdzone", DriverLeaveRequestUiRules.eventActionLabel("cancellation_approved"))
|
||||
assertEquals("Anulowanie odrzucone", DriverLeaveRequestUiRules.eventActionLabel("cancellation_rejected"))
|
||||
assertEquals("Aktualizacja wniosku", DriverLeaveRequestUiRules.eventActionLabel("some_raw_action"))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun driverCanCancelPendingAndFutureApprovedRequests() {
|
||||
val tomorrow = LocalDate.now().plusDays(1).toString()
|
||||
val yesterday = LocalDate.now().minusDays(1).toString()
|
||||
|
||||
assertTrue(DriverLeaveRequestUiRules.canCancel(status = "pending", dateFrom = yesterday))
|
||||
assertTrue(DriverLeaveRequestUiRules.canCancel(status = "approved", dateFrom = tomorrow))
|
||||
assertFalse(DriverLeaveRequestUiRules.canCancel(status = "approved", dateFrom = yesterday))
|
||||
assertFalse(DriverLeaveRequestUiRules.canCancel(status = "rejected", dateFrom = tomorrow))
|
||||
}
|
||||
}
|
||||
@@ -61,6 +61,30 @@ class DriverUiRulesTest {
|
||||
assertEquals(3, inlinePhotoGridRows(5))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun keepsPhotoPreviewFeedbackVisibleUntilImageLoadsOrFails() {
|
||||
assertTrue(shouldShowPhotoPreviewLoadingState(hasPhoto = true, isLoading = true, isError = false))
|
||||
assertFalse(shouldShowPhotoPreviewLoadingState(hasPhoto = true, isLoading = false, isError = false))
|
||||
assertFalse(shouldShowPhotoPreviewLoadingState(hasPhoto = true, isLoading = true, isError = true))
|
||||
assertFalse(shouldShowPhotoPreviewLoadingState(hasPhoto = false, isLoading = true, isError = false))
|
||||
|
||||
assertTrue(shouldShowPhotoPreviewErrorState(hasPhoto = true, isError = true))
|
||||
assertFalse(shouldShowPhotoPreviewErrorState(hasPhoto = true, isError = false))
|
||||
assertFalse(shouldShowPhotoPreviewErrorState(hasPhoto = false, isError = true))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun keepsPhotoTileFeedbackVisibleUntilThumbnailLoadsOrFails() {
|
||||
assertTrue(shouldShowPhotoTileLoadingState(hasPhoto = true, isLoading = true, isError = false))
|
||||
assertFalse(shouldShowPhotoTileLoadingState(hasPhoto = true, isLoading = false, isError = false))
|
||||
assertFalse(shouldShowPhotoTileLoadingState(hasPhoto = true, isLoading = true, isError = true))
|
||||
assertFalse(shouldShowPhotoTileLoadingState(hasPhoto = false, isLoading = true, isError = false))
|
||||
|
||||
assertTrue(shouldShowPhotoTileErrorState(hasPhoto = true, isError = true))
|
||||
assertFalse(shouldShowPhotoTileErrorState(hasPhoto = true, isError = false))
|
||||
assertFalse(shouldShowPhotoTileErrorState(hasPhoto = false, isError = true))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun usesShorterChromeOnCompactScreens() {
|
||||
assertEquals(118, authBrandBannerHeightDp(600))
|
||||
@@ -199,6 +223,22 @@ class DriverUiRulesTest {
|
||||
assertFalse(shouldShowDispatchSheetReminderCard(null))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun explainsOfflineStateWithoutCachedSync() {
|
||||
assertEquals(
|
||||
"Brak połączenia z serwerem. Sprawdź internet i spróbuj ponownie.",
|
||||
offlineStaleBannerMessage(isOnline = false, isStale = false, syncLabel = "brak zapisanej synchronizacji"),
|
||||
)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun explainsStaleCachedDataWhenOnline() {
|
||||
assertEquals(
|
||||
"Dane mogą być nieaktualne. Ostatnia synchronizacja: 12:30.",
|
||||
offlineStaleBannerMessage(isOnline = true, isStale = true, syncLabel = "12:30"),
|
||||
)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun labelsDispatchSheetActionByStatusAndLocalUploadQueue() {
|
||||
assertEquals("Zrób zdjęcie", dispatchSheetPrimaryActionLabel(dispatchReminder(status = "missing"), hasQueuedUpload = false))
|
||||
|
||||
@@ -3,6 +3,8 @@ package pl.firmatpp.kierowca.ui
|
||||
import org.junit.Assert.assertEquals
|
||||
import org.junit.Assert.assertTrue
|
||||
import org.junit.Test
|
||||
import pl.firmatpp.kierowca.data.model.DriverLeaveRequestDto
|
||||
import pl.firmatpp.kierowca.ui.theme.AppThemeMode
|
||||
|
||||
class DriverUiStateTest {
|
||||
@Test
|
||||
@@ -12,4 +14,49 @@ class DriverUiStateTest {
|
||||
assertEquals(DriverScreen.Initializing, state.screen)
|
||||
assertTrue(state.loading)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun startsWithMaterial3ThemeMode() {
|
||||
val state = DriverUiState()
|
||||
|
||||
assertEquals(AppThemeMode.Material3, state.themeMode)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun refreshingLeaveRequestsKeepsDetailScreenOpenAndUpdatesSelectedRequest() {
|
||||
val state = DriverUiState(
|
||||
screen = DriverScreen.LeaveRequestDetail,
|
||||
selectedLeaveRequest = leaveRequest(id = "leave-1", status = "pending"),
|
||||
)
|
||||
|
||||
val refreshed = state.withLoadedLeaveRequests(
|
||||
requests = listOf(leaveRequest(id = "leave-1", status = "approved")),
|
||||
navigateToList = false,
|
||||
)
|
||||
|
||||
assertEquals(DriverScreen.LeaveRequestDetail, refreshed.screen)
|
||||
assertEquals("approved", refreshed.selectedLeaveRequest?.status)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun openingLeaveRequestsNavigatesToList() {
|
||||
val state = DriverUiState(screen = DriverScreen.Routes)
|
||||
|
||||
val refreshed = state.withLoadedLeaveRequests(
|
||||
requests = listOf(leaveRequest(id = "leave-1", status = "pending")),
|
||||
navigateToList = true,
|
||||
)
|
||||
|
||||
assertEquals(DriverScreen.LeaveRequests, refreshed.screen)
|
||||
assertEquals(1, refreshed.leaveRequests.size)
|
||||
}
|
||||
|
||||
private fun leaveRequest(id: String, status: String): DriverLeaveRequestDto =
|
||||
DriverLeaveRequestDto(
|
||||
id = id,
|
||||
dateFrom = "2026-07-10",
|
||||
dateTo = "2026-07-12",
|
||||
type = "URLOP",
|
||||
status = status,
|
||||
)
|
||||
}
|
||||
|
||||
@@ -0,0 +1,23 @@
|
||||
package pl.firmatpp.kierowca.ui.theme
|
||||
|
||||
import org.junit.Assert.assertEquals
|
||||
import org.junit.Test
|
||||
|
||||
class AppThemeModeTest {
|
||||
@Test
|
||||
fun parsesStoredMaterial3Value() {
|
||||
assertEquals(AppThemeMode.Material3, AppThemeMode.fromStoredValue("material3"))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun fallsBackToCurrentForUnknownStoredValue() {
|
||||
assertEquals(AppThemeMode.Material3, AppThemeMode.fromStoredValue("future-theme"))
|
||||
assertEquals(AppThemeMode.Material3, AppThemeMode.fromStoredValue(null))
|
||||
}
|
||||
|
||||
@Test
|
||||
fun storesStablePreferenceValues() {
|
||||
assertEquals("current", AppThemeMode.Current.storedValue)
|
||||
assertEquals("material3", AppThemeMode.Material3.storedValue)
|
||||
}
|
||||
}
|
||||
@@ -3,4 +3,5 @@ plugins {
|
||||
alias(libs.plugins.kotlin.android) apply false
|
||||
alias(libs.plugins.kotlin.compose) apply false
|
||||
alias(libs.plugins.google.services) apply false
|
||||
alias(libs.plugins.firebase.crashlytics) apply false
|
||||
}
|
||||
|
||||
@@ -20,6 +20,7 @@ playServicesAuthApiPhone = "18.3.0"
|
||||
junit = "4.13.2"
|
||||
firebaseBom = "33.7.0"
|
||||
googleServices = "4.4.2"
|
||||
firebaseCrashlyticsPlugin = "3.0.7"
|
||||
|
||||
[libraries]
|
||||
activity-compose = { group = "androidx.activity", name = "activity-compose", version.ref = "activityCompose" }
|
||||
@@ -55,6 +56,7 @@ retrofit = { group = "com.squareup.retrofit2", name = "retrofit", version.ref =
|
||||
retrofit-gson = { group = "com.squareup.retrofit2", name = "converter-gson", version.ref = "retrofit" }
|
||||
firebase-bom = { group = "com.google.firebase", name = "firebase-bom", version.ref = "firebaseBom" }
|
||||
firebase-messaging = { group = "com.google.firebase", name = "firebase-messaging-ktx" }
|
||||
firebase-crashlytics = { group = "com.google.firebase", name = "firebase-crashlytics" }
|
||||
|
||||
[plugins]
|
||||
android-application = { id = "com.android.application", version.ref = "agp" }
|
||||
@@ -62,3 +64,4 @@ kotlin-android = { id = "org.jetbrains.kotlin.android", version.ref = "kotlin" }
|
||||
kotlin-compose = { id = "org.jetbrains.kotlin.plugin.compose", version.ref = "kotlin" }
|
||||
ksp = { id = "com.google.devtools.ksp", version.ref = "ksp" }
|
||||
google-services = { id = "com.google.gms.google-services", version.ref = "googleServices" }
|
||||
firebase-crashlytics = { id = "com.google.firebase.crashlytics", version.ref = "firebaseCrashlyticsPlugin" }
|
||||
|
||||
Reference in New Issue
Block a user