fix(tasks): uncheck restores InProgress + create re-fetches by saved SID #144

Merged

zaelgohary merged 2 commits from development_fix_tasks_uncheck_status into development

2026-04-23 08:48:26 +00:00

zaelgohary commented

2026-04-23 08:27:09 +00:00

Member

Summary

Two small fixes to the Tasks island. Unchecking a Done task used to drop it back to Todo, erasing that the task had ever been started; it now restores InProgress. And creating a task would surface a red "Method not found" banner even after the task saved — the service was re-fetching by the "0000" placeholder SID from task.new instead of the real SID task.set returned.

Closes #138

Changes

Unchecking a completed task restores InProgress instead of Todo, so in-flight work isn't quietly thrown away
Creating a task no longer flashes a "Method not found" error — the service now reads back the saved task by the SID the server returned

Test Results

cargo check / clippy -D warnings / fmt --check clean on hero_archipelagos_tasks; Playwright covers both transitions end-to-end against the shared test instance.

Manual verification

Mark a task as Done, then uncheck it — status lands on InProgress (visible in the task detail view)
Create a new task — the row appears in the list with no red banner

## Summary Two small fixes to the Tasks island. Unchecking a Done task used to drop it back to Todo, erasing that the task had ever been started; it now restores InProgress. And creating a task would surface a red "Method not found" banner even after the task saved — the service was re-fetching by the `"0000"` placeholder SID from `task.new` instead of the real SID `task.set` returned. ## Related Issue Closes #138 ## Changes - Unchecking a completed task restores InProgress instead of Todo, so in-flight work isn't quietly thrown away - Creating a task no longer flashes a "Method not found" error — the service now reads back the saved task by the SID the server returned ## Test Results `cargo check / clippy -D warnings / fmt --check` clean on `hero_archipelagos_tasks`; Playwright covers both transitions end-to-end against the shared test instance. ## Manual verification - [ ] Mark a task as Done, then uncheck it — status lands on InProgress (visible in the task detail view) - [ ] Create a new task — the row appears in the list with no red banner

zaelgohary added 2 commits

2026-04-23 08:27:09 +00:00