Switching day tabs silently discards unsaved schedule edits (TODO in SchedulePage)

オープン
  • 何か問題がありますか？
#59 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
58/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
停滞
技術スタック
react, typescript
領域
frontend

調査の方向性

Read ui/src/pages/SchedulePage/SchedulePage.tsx around line 55, then trace selectDay through scheduleStore.tsx lines 67–88 and the tab handlers in DayTabs.tsx line 16. Check how changesPresent is set and cleared, then reproduce switching days or sides with unsaved power time, temperature, or alarm edits. Done means those edits are not silently discarded and the user gets a confirm prompt before they are lost.

索引モデルが issue の本文から書いたものです。

説明

enhancement

ui/src/pages/SchedulePage/SchedulePage.tsx:55 notes // TODO: Add changes lost notification using changesPresent when user tries to switch tab before saving. selectDay -> reloadScheduleData (scheduleStore.tsx:67-88) resets selectedSchedule from originalSchedules and sets changesPresent:false with no warning, so unsaved edits to power time/temperature/alarm are lost when the user taps another day tab (DayTabs.tsx:16) or switches side. The store already tracks changesPresent; wiring a confirm prompt would prevent silent data loss. Related: #6 (Scheduling).

Filed from a repo-wide code audit (working tree at bcad746).

主要言語
Rust
スター
0
フォーク
0
30日以内にマージされた PR はありません

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

GoogleBot42/podd のほかの issue

GoogleBot42/podd の issue をすべて見る

似ている issue

Rust の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。