From 0eed12d18ebc4b68b574dd017c88ab8f9b447bf8 Mon Sep 17 00:00:00 2001 From: M57 <41645758+Hy4ri@users.noreply.github.com> Date: Sun, 21 Jun 2026 00:17:37 +0300 Subject: [PATCH] Refactor: Extract hardcoded "todo" to constant in KanbanScreen (#190) - Extracted the hardcoded string "todo" to a `DEFAULT_COLUMN` constant in `KanbanScreen.kt` to improve code health and readability. - Replaced the hardcoded string literal in the `viewModel.createTask` call with the new constant. --- .../main/java/com/m57/hermescontrol/ui/kanban/KanbanScreen.kt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/com/m57/hermescontrol/ui/kanban/KanbanScreen.kt b/app/src/main/java/com/m57/hermescontrol/ui/kanban/KanbanScreen.kt index 206044c4..b903be4c 100644 --- a/app/src/main/java/com/m57/hermescontrol/ui/kanban/KanbanScreen.kt +++ b/app/src/main/java/com/m57/hermescontrol/ui/kanban/KanbanScreen.kt @@ -48,6 +48,8 @@ import com.m57.hermescontrol.ui.common.ErrorState import com.m57.hermescontrol.ui.common.HermesScaffold import com.m57.hermescontrol.ui.common.LoadingState +private const val DEFAULT_COLUMN = "todo" + @OptIn(ExperimentalMaterial3Api::class) @Composable fun KanbanScreen( @@ -185,7 +187,7 @@ fun KanbanScreen( AddTaskDialog( onDismiss = { showAddTaskDialog = false }, onConfirm = { title, desc -> - viewModel.createTask(title, desc, state.columns.firstOrNull()?.name ?: "todo") + viewModel.createTask(title, desc, state.columns.firstOrNull()?.name ?: DEFAULT_COLUMN) showAddTaskDialog = false }, )