Skip to content

Comments

家事承認機能の実装#68

Merged
stotic-dev merged 20 commits intomainfrom
update/impl_approve_housework
Jan 31, 2026
Merged

家事承認機能の実装#68
stotic-dev merged 20 commits intomainfrom
update/impl_approve_housework

Conversation

@stotic-dev
Copy link
Owner

@stotic-dev stotic-dev commented Jan 17, 2026

Summary

  • 家事の承認・未完了戻し機能を実装
  • 家族グループメンバー情報を管理するCohabitantStoreを追加
  • HouseworkItemに承認関連プロパティ(reviewerId, approvedAt, reviewerComment)を追加
  • 承認時にパートナーへプッシュ通知を送信する機能を追加
  • 各機能のユニットテストを追加

関連Issue: #64

主な変更内容

ドメインモデル

  • HouseworkItem: 承認関連プロパティと状態更新メソッド(updatePendingApproval, updateApproved, updateIncomplete, canReview)を追加
  • CohabitantStore: 家族グループメンバー情報を監視・キャッシュするStoreを新規作成
  • CohabitantMemberList, CohabitantMember: メンバー情報を管理するモデルを追加
  • PushNotificationContent: 承認通知用のファクトリーメソッドを追加

UI

  • HouseworkApprovalView: 承認ボタンタップで家事を完了状態に更新
  • HouseworkDetailActionContent: 未完了に戻すボタンの機能を実装
  • HouseworkDetailItemListContent: 実施者名を表示するコンポーネントを追加

テスト

  • HouseworkItemTest: canReview、状態更新メソッドのテストを追加
  • HouseworkIndexedDateTest: calcTargetPeriodのテストを追加
  • HouseworkListStoreTest: approved、returnToIncompleteのテストを追加
  • CohabitantStoreTest: メンバー監視のテストを追加

Test plan

  • 家事詳細画面で「確認してもらう」ボタンをタップして承認待ち状態にできること
  • 承認待ち状態の家事を別のユーザーが承認して完了状態にできること
  • 承認時にパートナーへプッシュ通知が送信されること
  • 「未完了に戻す」ボタンで家事を未完了状態に戻せること
  • 自分が実施した家事は自分でレビューできないこと
  • ユニットテストが全てパスすること

🤖 Generated with Claude Code

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

Warnings
⚠️ PRの変更行が多すぎます。500行以内にしてね!理想は400行!
⚠️

homete/Model/Domain/Cohabitant/Housework/HouseworkListStore.swift#L72 - TODOs should be resolved (PushNotificationContentにファクトリー...) (todo)

⚠️

homete/Views/HouseworkApproval/HouseworkApprovalView.swift#L113 - TODOs should be resolved (家事を未完了に戻す) (todo)

⚠️

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)

Messages
📖 Project coverage: 43.67%

homete.app: Coverage: 56.5

File Coverage
HouseworkDetailItemListContent.swift 100.0%
DailyHouseworkMetaData.swift 85.71%
HouseworkDetailActionContent.swift 61.48%
HouseworkUtil.swift 100.0%
HouseworkIndexedDate.swift 90.91%
HouseworkListStore.swift 89.24%
HouseBoardListRow.swift 100.0%
PushNotificationContent.swift 100.0%
HouseworkApprovalView.swift 88.19%
HouseworkBoardView.swift 77.65%
HouseworkBoardList.swift 93.75%
HouseworkItemPropertyListContent.swift 100.0%
HouseworkBoardListContent.swift 72.15%
HouseworkItem.swift 100.0%
HouseworkDetailView.swift 63.27%
DailyHouseworkList.swift 100.0%
HouseworkClient.swift 12.5%
RegisterHouseworkView.swift 74.48%

hometeTests.xctest: Coverage: 99.22

File Coverage
HouseworkIndexedDate.swift 100.0%
DailyHouseworkListTest.swift 100.0%
CalendarHelper.swift 100.0%
HouseworkListStoreTest.swift 99.42%
HouseworkItemTest.swift 100.0%
HouseworkItemHelper.swift 97.92%
HouseworkBoardListTest.swift 100.0%

Generated by 🚫 Danger Swift against 525711d

@stotic-dev stotic-dev merged commit bc4b555 into main Jan 31, 2026
4 checks passed
@stotic-dev stotic-dev deleted the update/impl_approve_housework branch January 31, 2026 06:51
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