Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 38bb2eb628
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| }; | ||
| /** 관리자 계정 단건 조회 */ | ||
| get: operations['getUserById']; | ||
| put?: never; |
There was a problem hiding this comment.
Remove conflicting
put type in admin user detail path
The new OpenAPI path type declares put twice for /api/admin/user/{id} (put?: never here and put: operations['putUser'] later), which makes the declaration internally contradictory. This can break strict type-checking/tooling (for example when skipLibCheck is off) and makes method inference for this endpoint unreliable; keep only the concrete put operation declaration.
Useful? React with 👍 / 👎.
Summary
관리자 계정 관리 화면과 관련 API controller를
apps/admin에 추가했습니다.관리자 계정 목록 조회, 생성, 수정, 삭제 흐름을 admin UI에서 처리할 수 있도록 연결했습니다.
메뉴 항목이 늘어나면서 사이드바 하단이 잘리던 문제도 함께 수정해, 메뉴 영역만 스크롤되도록 정리했습니다.
Linear
Changes
/admin-user) 추가 및 생성/수정/삭제 UI 구현Testing
pnpm lintpnpm typecheckRisk / Impact
apps/admin의 관리자 계정 관리 화면, GNB 메뉴, user API wrapper/api/admin/user,/api/admin/user/{id}스펙이 현재 schema와 동일해야 함Screenshots / Video