Skip to content

docs: グループメンバー一覧機能の実装方針ドキュメントを追加#160

Open
stotic-dev wants to merge 2 commits into
mainfrom
feat/group_member_list
Open

docs: グループメンバー一覧機能の実装方針ドキュメントを追加#160
stotic-dev wants to merge 2 commits into
mainfrom
feat/group_member_list

Conversation

@stotic-dev
Copy link
Copy Markdown
Owner

経緯

Issue #154 のグループメンバー一覧画面の実装にあたり、要件・設計方針・タスクを整理する必要があったため。
実装着手前に方針を明文化し、進捗管理可能な形で残すことで、レビューや並行作業時の認識合わせを容易にする。

実装内容

doc/strategy/group-member-list.md を新規追加。既存の doc/strategy/housework-points.md の構成を参考に、以下の観点で内容を整理した。

  • 進捗管理しやすい構成にした理由: チェックボックス形式のステータスと Phase 別タスクを設けることで、レビュー時にどこまで合意が取れているか・何が残っているかを一目で把握できるようにした
  • 設計確定事項を明記した理由: 「自分以外のメンバー取得方法」と「SubView粒度」について事前に方針を確定させ、実装フェーズで手戻りが発生しないようにした
    • CohabitantContext を新設し、グループID・メンバー一覧・othersをまとめてEnvironmentでDIする方針を採用
    • SubView は GroupMemberListView + GroupMemberRow に分離する方針を採用
  • 既存影響範囲を明示した理由: \.cohabitantMembers Environment は ContributionFeature 等でも参照されているため、置き換えに伴う修正対象をファイル配置表に列挙し、抜け漏れを防いだ

確認内容

  • 要件(Issue グループメンバーの一覧画面 #154)と記載内容に齟齬がないこと
  • 設計方針セクションが、実装着手時に十分な情報を提供していること
  • タスクリストが Phase 1〜4 で進捗管理可能な形になっていること
  • 既存ファイルへの参照リンクが正しいパスを指していること

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant