グループ管理画面のグループ作成・編集・メンバー一覧取得機能を実装する: frontend - #240
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
グループ管理画面の「グループ作成・編集・削除」と「メンバー一覧取得」を、ダミーデータから実 API 呼び出しへ置き換え、編集時は差分メンバーのみ送る形に拡張するPRです(#236対応)。
Changes:
- frontend: グループ作成/編集/削除用の React Query mutation を追加し、画面から API を呼ぶように変更
- frontend: メンバー一覧取得を
/groups/users?groupId=...へ接続し、型定義を API 仕様に合わせて更新 - backend: グループ更新 API(親/名前 + メンバー変更)を facade でまとめ、関連 DTO/Repository/Service を更新
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/features/management/types/member.ts | メンバー型・更新リクエスト型を API 仕様に合わせて変更 |
| frontend/src/features/management/types/group.ts | 作成/削除リクエスト型の整理(命名変更・削除用型追加) |
| frontend/src/features/management/hooks/useMember.ts | メンバー取得を JWT + api-client 経由の実 API 呼び出しへ移行 |
| frontend/src/features/management/hooks/useGroups.ts | グループツリーの placeholderData をダミーから空配列へ |
| frontend/src/features/management/hooks/useEditGroup.ts | グループ編集 mutation 追加(差分メンバー送信) |
| frontend/src/features/management/hooks/useDeleteGroup.ts | グループ削除 mutation 追加(クエリ無効化含む) |
| frontend/src/features/management/hooks/useCreateGroup.ts | グループ作成 mutation 追加(クエリ無効化含む) |
| frontend/src/app/routes/app/management/group/groupNew.tsx | 作成画面を mutation 呼び出しへ接続、スピナー追加 |
| frontend/src/app/routes/app/management/group/groupEdit.tsx | 編集/削除を mutation 呼び出しへ接続、差分抽出・スピナー追加 |
| backend/src/main/java/com/example/backend/group/service/Impl/GroupServiceImpl.java | modifyUpperGroup→modifyGroup へ置換し helper 呼び出し変更 |
| backend/src/main/java/com/example/backend/group/service/GroupService.java | グループ更新メソッド名・DTO を更新 |
| backend/src/main/java/com/example/backend/group/repository/GroupRepository.java | 親ID更新クエリを「親ID+名前更新」に変更 |
| backend/src/main/java/com/example/backend/group/helper/GroupHelper.java | 親ID/名前の更新をまとめた helper メソッドに変更 |
| backend/src/main/java/com/example/backend/group/facade/ModifyGroupFacade.java | 親/名前更新 + メンバー更新を1トランザクションで実行 |
| backend/src/main/java/com/example/backend/group/dto/ModifyGroupRequest.java | グループ編集の統合リクエスト DTO を追加 |
| backend/src/main/java/com/example/backend/group/dto/ModifyGroupInfoRequest.java | DTO 名称変更 + グループ名更新項目追加 |
| backend/src/main/java/com/example/backend/group/dto/GroupCreateRequest.java | 作成 DTO の membersUserId 初期化/バリデーションを変更 |
| backend/src/main/java/com/example/backend/group/controller/GroupController.java | delete を RequestParam 化、統合 modify エンドポイント追加 |
| backend/src/main/java/com/example/backend/accounts/service/impl/StudentServiceImpl.java | groupId=0 のユーザ一覧取得ロジックを調整 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
対象イシュー
イシューリンク
#236
イシュー番号(自動クローズ用)
close #236
やったこと
やらないこと
できるようになること(ユーザ目線)
できなくなること(ユーザ目線)
動作確認
影響範囲
テスト
備考