feat: ユーザー複数部門管理機能を実装(1/2) #1017
Closed
Conversation
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
- 部門管理ページの新規作成 - 部門の一覧、追加、編集、削除機能 - 管理者権限チェック - モックAPIで動作確認 - ユーザー管理の複数部門対応 - ユーザー一覧で複数部門をタグ表示 - ユーザー編集モーダルで部門の追加/削除が可能 - 直感的なUI/UXで部門管理 - ナビゲーション改善 - 管理セクションをサイドバーに追加 - 部門管理とユーザー管理へのアクセス 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
There was a problem hiding this comment.
Pull Request Overview
複数部門に所属できるユーザー管理機能を実装するPRです。部門管理ページの新規作成と既存ユーザー管理機能の拡張を行いました。
- 新しい部門管理ページの実装（CRUD操作対応）
- ユーザー管理での複数部門所属機能の追加
- ナビゲーションに管理セクションを追加
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| pages/users/index.tsx | ユーザー一覧で複数部門をタグ表示、モックデータへの変更 |
| pages/divisions/index.tsx | 部門管理ページの新規作成（一覧・CRUD機能） |
| constants/linkItem.tsx | ナビゲーションに管理セクション追加 |
| components/users/OpenEditModalButton.tsx | 部門情報をpropsに追加 |
| components/users/EditModal.tsx | ユーザー編集で複数部門管理機能を実装 |
| components/divisions/OpenEditModalButton.tsx | 部門編集モーダルボタンコンポーネント |
| components/divisions/OpenDeleteModalButton.tsx | 部門削除モーダルボタンコンポーネント |
| components/divisions/OpenAddModalButton.tsx | 部門追加モーダルボタンコンポーネント |
| components/divisions/EditModal.tsx | 部門編集モーダルコンポーネント |
| components/divisions/DeleteModal.tsx | 部門削除モーダルコンポーネント |
| components/divisions/AddModal.tsx | 部門追加モーダルコンポーネント |
| components/common/SideNav.tsx | サイドナビゲーションに管理セクション追加 |
Comment on lines
+15
to
+23
| interface Division { | ||
| id: number; | ||
| name: string; | ||
| } | ||
|
|
||
| interface UserWithDivisions extends Omit<User, 'bureauID'> { | ||
| bureauID?: number; | ||
| divisions?: Division[]; | ||
| } |
There was a problem hiding this comment.
めちゃ同じDivisionインターフェース定義してるじゃん！！😱 複数ファイルで重複してるのやばたにえんよ〜💦 共通の型定義ファイルに移動してリスペクトしかない！✨
Comment on lines
+75
to
+84
| console.log('Updating user with divisions:', { | ||
| id, | ||
| name: data.name, | ||
| bureauID: data.bureauID, | ||
| roleID: data.roleID, | ||
| divisions: data.divisions | ||
| }); | ||
|
|
||
| // 成功時の処理 | ||
| alert('ユーザーの部門を更新しました'); |
There was a problem hiding this comment.
console.logとalertがあるのは開発中だからわかるけど、プロダクションコードには適切なエラーハンドリングとトースト通知を実装した方がいいぞい〜🚀 ユーザー体験がしぬほど良くなるよ！💕
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.
対応Issue
新機能実装のためissueなし
概要
ユーザーが複数の部門に所属できる機能を実装しました。部門管理ページの新規作成とユーザー管理機能の拡張を行いました。
主な実装内容
1. 部門管理ページの新規作成
/pages/divisions/index.tsx- 部門一覧ページ2. ユーザー管理の複数部門対応
3. ナビゲーション改善
技術的詳細
画面スクリーンショット等
部門管理ページ
ユーザー管理ページ
テスト項目
備考
🤖 Generated with Claude Code
Co-Authored-By: Claude noreply@anthropic.com