Calendar island: color-picker panics on duplicate keys, event list is N+1, delete has no confirm #108

Open
opened 2026-04-21 15:23:58 +00:00 by zaelgohary · 0 comments
Member

Bugs

  • Color picker crashes the app. The list keys each option by its hex value, but two of the predefined colors are the same #007AFF. Dioxus 0.7 panics on duplicate keys (same class we hit in #101) and the WASM goes unreachable.
  • Every mount hits the backend N times. event_list fans out into a sequential for sid in sids { event_get().await }. With 100 events that's 100 serial round-trips.
  • Delete has no confirmation. One click on the trash icon and the event is gone.

Extras (lower priority)

Date-parse fallbacks silently swap today's date in on bad input, and a couple of NaiveTime::from_hms_opt(...).unwrap() on constants — all safe today but worth tidying next to the above.

### Bugs - **Color picker crashes the app.** The list keys each option by its hex value, but two of the predefined colors are the same `#007AFF`. Dioxus 0.7 panics on duplicate keys (same class we hit in #101) and the WASM goes `unreachable`. - **Every mount hits the backend N times.** `event_list` fans out into a sequential `for sid in sids { event_get().await }`. With 100 events that's 100 serial round-trips. - **Delete has no confirmation.** One click on the trash icon and the event is gone. ### Extras (lower priority) Date-parse fallbacks silently swap today's date in on bad input, and a couple of `NaiveTime::from_hms_opt(...).unwrap()` on constants — all safe today but worth tidying next to the above.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_archipelagos#108
No description provided.