Skip to content

Comments

家事の再確認依頼(却下)機能を実装#74

Merged
stotic-dev merged 5 commits intomainfrom
update/impl_reject_housework
Feb 14, 2026
Merged

家事の再確認依頼(却下)機能を実装#74
stotic-dev merged 5 commits intomainfrom
update/impl_reject_housework

Conversation

@stotic-dev
Copy link
Owner

@stotic-dev stotic-dev commented Feb 7, 2026

Summary

  • 家事承認画面で「再確認してもらう」ボタンを押すと、家事を却下(再確認依頼)できる機能を実装
  • ステータス更新処理の重複コードを共通ヘルパー関数にリファクタリング
  • プッシュ通知メッセージをファクトリーメソッドに統一

関連Issue: #70

Background

家事の承認フローにおいて、レビュアーが家事の完了を承認するだけでなく、再確認を依頼(却下)できる機能が必要でした。これにより、家事の完了状態に問題がある場合に実行者へフィードバックできるようになります。

Changes

  • HouseworkItem: updateRejected() メソッドを追加(却下時のステータス更新)
  • HouseworkListStore: rejected() メソッドを新規追加、requestReview()/approved()/returnToIncomplete() を共通の updateAndSave() ヘルパーで共通化、returnToIncomplete() から不要な now パラメータを削除
  • PushNotificationContent: addNewHouseworkItem()/requestReviewMessage()/rejectedMessage() のファクトリーメソッドを追加
  • HouseworkApprovalView: 「未完了に戻す」を「再確認してもらう」に変更、ローディングインジケータ追加
  • StoredAllHouseworkList: item(::) のAPIを item(_:) に簡素化
  • テスト: rejected() のユニットテストを追加、UpdateStatusCase ネストstructでテストを整理

Test plan

  • 家事承認画面でコメントを入力し「再確認してもらう」ボタンを押して、家事が未完了に戻ることを確認
  • 却下時にパートナーへプッシュ通知が送信されることを確認
  • 承認ボタンの既存動作に影響がないことを確認
  • ユニットテストが全てパスすることを確認
  • スナップショットテストが更新されていることを確認

🤖 Generated with Claude Code

@stotic-dev stotic-dev self-assigned this Feb 7, 2026
@stotic-dev stotic-dev added the feature 機能追加 label Feb 7, 2026
@stotic-dev-bot
Copy link
Collaborator

Warnings
⚠️

homete/Views/HouseworkDetailView/SubViews/HouseworkDetailActionContent.swift#L25 - Include a single empty line between switch cases (vertical_whitespace_between_cases)

⚠️

homete/Views/HouseworkDetailView/SubViews/HouseworkDetailActionContent.swift#L32 - Include a single empty line between switch cases (vertical_whitespace_between_cases)

⚠️

hometeTests/Domain/Housework/HouseworkListStoreTest.swift#L415 - File should contain 400 lines or less: currently contains 415 (file_length)

Messages
📖 Project coverage: 29.15%

snapshotの変更

更新ファイル: HouseworkApprovalView_0-iPhone-16-Pro-Max.1.png

before after
image image

更新ファイル: HouseworkApprovalView_0-iPhone-16.1.png

before after
image image

homete.app: Coverage: 56.63

File Coverage
HouseworkDetailActionContent.swift 61.48%
HouseworkListStore.swift 93.29%
PushNotificationContent.swift 100.0%
HouseworkApprovalView.swift 81.85%
StoredAllHouseworkList.swift 91.43%
HouseworkItem.swift 100.0%
HouseworkDetailView.swift 63.27%

hometeTests.xctest: Coverage: 99.28

File Coverage
StoredAllHouseworkListTest.swift 100.0%
HouseworkListStoreTest.swift 99.54%

Generated by 🚫 Danger Swift against e576845

@stotic-dev stotic-dev merged commit bcca17e into main Feb 14, 2026
6 of 7 checks passed
@stotic-dev stotic-dev deleted the update/impl_reject_housework branch February 14, 2026 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 機能追加

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants