Skip to content

Comments

退会機能の実装_退会を検知したら紐づくユーザーデータ削除処理実装#58

Merged
stotic-dev merged 14 commits intomainfrom
update/impl_delete_user_data_api_function
Jan 3, 2026
Merged

退会機能の実装_退会を検知したら紐づくユーザーデータ削除処理実装#58
stotic-dev merged 14 commits intomainfrom
update/impl_delete_user_data_api_function

Conversation

@stotic-dev
Copy link
Owner

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

経緯

関連Issue: #56

実装内容

Accountの情報にcohabitantIdを追加しました。
これは、アカウントに紐づく情報で、アカウント後に情報をスイッチする必要があるのと、ユーザーデータ削除時にも、効率的に紐づくCohabitant情報を取得するためです。
Functionsにユーザー削除をトリガーに紐づくFirestoreのAccountとCohabitantの情報を削除するAPIを追加しました。
Cohabitantは参加者が2名の場合はCohabitantのドキュメント毎削除し、3人以上の場合は、今後も残った二人で家事管理が可能なのでCohabitant/membersから該当のユーザーだけ削除します。
また、ユーザー削除のAPIのE2Eテストを実装し、これを検証するCIワークフローを追加しました。

確認内容

  • CIが全て通ること
  • パートナー登録まで住んでいるアカウントの退会を行い、紐づくAccount、Cohabitantの情報が削除されていること
  • パートナー登録まで行っていないアカウントの退会を行い、紐づくAccountのみ削除されていること

@stotic-dev stotic-dev self-assigned this Jan 3, 2026
@stotic-dev stotic-dev added feature 機能追加 mentenance 開発環境のメンテナンス labels Jan 3, 2026
@stotic-dev-bot
Copy link
Collaborator

stotic-dev-bot commented Jan 3, 2026

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

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)

⚠️

homete/Views/HouseworkDetailView/SubViews/HouseworkDetailActionContent.swift#L58 - TODOs should be resolved (未完了に戻す) (todo)

⚠️

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

⚠️

homete/Views/RootView/RootView.swift#L31 - Include a single empty line between switch cases (vertical_whitespace_between_cases)

⚠️

homete/Views/RootView/RootView.swift#L35 - Include a single empty line between switch cases (vertical_whitespace_between_cases)

⚠️

homete/Views/SettingView/SettingView.swift#L32 - TODOs should be resolved (メニューボタンタップ時の処理) (todo)

⚠️

homete/Views/SettingView/SettingView.swift#L119 - TODOs should be resolved (エラーハンドリング) (todo)

Messages
📖 Project coverage: 34.25%

homete.app: Coverage: 54.81

File Coverage
Account.swift 0.0% ⚠️
LaunchState.swift 100.0%
onCompleteCohabitantRegistration.swift 0.0% ⚠️
HouseworkDetailActionContent.swift 68.47%
AppTabView.swift 60.31%
CohabitantRegistrationProcessingFollowerView.swift 0.0% ⚠️
HomeView.swift 89.41%
AccountStore.swift 78.13%
LoginView.swift 69.12%
CohabitantRegistrationPeersListView.swift 79.5%
AppStorageExtensions.swift 82.61%
LoginContext.swift 100.0%
SettingView.swift 81.58%
CohabitantRegistrationView.swift 0.0% ⚠️
CohabitantRegistrationProcessingLeader.swift 0.0% ⚠️
RootView.swift 71.19%

hometeTests.xctest: Coverage: 99.06

File Coverage
AccountStoreTest.swift 100.0%

Generated by 🚫 Danger Swift against 5edd23c

@stotic-dev stotic-dev merged commit 49b644e into main Jan 3, 2026
5 checks passed
@stotic-dev stotic-dev deleted the update/impl_delete_user_data_api_function branch January 3, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 機能追加 mentenance 開発環境のメンテナンス

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants