admin multiselect#38
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a multi-select feature for university participation registration, allowing administrators to draft and register multiple entries at once. The changes include a new UniversityMultiSelect component and updated hooks to manage draft rows and bulk creation. Feedback focuses on removing deprecated state and mutations, addressing potential data truncation issues due to hardcoded page sizes, and preventing registration actions while initial data is still loading to ensure accurate duplicate checks.
| const [selectedUniversityId, setSelectedUniversityId] = useState(''); | ||
| const [teamName, setTeamName] = useState(''); |
There was a problem hiding this comment.
UniversityMultiSelect への移行に伴い、selectedUniversityId および teamName ステートは使用されなくなっています。また、これらを使用していた createMutation も不要になっているため、コードのクリーンアップとしてこれらを削除することをお勧めします。
| queryFn: async () => { | ||
| const result = await apiClient.GET('/api/admin/editions/{id}/participations', { | ||
| params: { path: { id: editionId } }, | ||
| params: { path: { id: editionId }, query: { pageSize: 100 } }, |
There was a problem hiding this comment.
pageSize: 100 が指定されていますが、出場登録数が100件を超える場合、リストに表示されないデータが発生します。また、createManyMutation 内の重複チェック（existingKeys）も取得済みの100件のみを対象とするため、それ以降のデータとの重複を検知できません。将来的に無限スクロールやページネーションの導入を検討してください。
| </div> | ||
| <Button | ||
| onClick={() => createManyMutation.mutate()} | ||
| disabled={draftRows.length === 0 || createManyMutation.isPending} |
There was a problem hiding this comment.
isLoading が true の間（既存の出場登録データを取得中）は、createManyMutation で使用される既存データの重複チェックが正しく機能しません。データの読み込みが完了するまで「登録」ボタンを無効化することをお勧めします。
| disabled={draftRows.length === 0 || createManyMutation.isPending} | |
| disabled={draftRows.length === 0 || createManyMutation.isPending || isLoading} |
1 participant
No description provided.