軽微なコード整理 #7

Closed
opened 2026-09-27 10:15:10 +00:00 by joe · 1 comment
Owner

レビューで挙がった軽微な整理事項:

  • app/routes/estimates.py _parse_items(estimate)の未使用引数estimateを削除
  • app/templates/dashboard.html 香典合計の桁区切りを他画面と統一(テストtest_dashboard_summaryが素の数値を期待しているのでテスト側も調整)
  • OCR失敗時、保存済み画像ファイル名を/expenses/new?receipt_filename=...に引き渡して孤立を防ぐ(new_expenseは既にv=dict(request.args)で受けられる)
レビューで挙がった軽微な整理事項: - `app/routes/estimates.py` `_parse_items(estimate)`の未使用引数`estimate`を削除 - `app/templates/dashboard.html` 香典合計の桁区切りを他画面と統一(テスト`test_dashboard_summary`が素の数値を期待しているのでテスト側も調整) - OCR失敗時、保存済み画像ファイル名を`/expenses/new?receipt_filename=...`に引き渡して孤立を防ぐ(`new_expense`は既に`v=dict(request.args)`で受けられる)
Author
Owner

対応しました。

  • _parse_items の未使用引数を削除
  • ダッシュボードの香典合計・経費合計を桁区切り表示に統一(テスト側を修正)
  • OCR失敗時、保存済み画像のファイル名を登録フォームへ引き継ぐように(画像が孤立しない)
対応しました。 - `_parse_items` の未使用引数を削除 - ダッシュボードの香典合計・経費合計を桁区切り表示に統一(テスト側を修正) - OCR失敗時、保存済み画像のファイル名を登録フォームへ引き継ぐように(画像が孤立しない)
joe closed this issue 2026-09-27 11:04:28 +00:00
Sign in to join this conversation.
No labels
bug
cleanup
enhancement
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
joe/sougi#7
No description provided.