Repository navigation
feat: 계정 아이디에 영문 대문자 허용 - #73
Conversation
BE #526과 같은 규칙으로 관리자·판매자 생성 폼의 아이디 검증을 완화함.
- 정규식: [a-z0-9._-]{4,80} → [A-Za-z0-9._-]{4,80}(create-admin-schema·create-seller-schema)
- 오류 문구: "영문 소문자" → "영문"
- 도움말: 로그인할 때 대소문자를 구분하지 않는다는 안내 추가(BE 컬럼 정렬이 ci라 대소문자만 다른 아이디는 같은 아이디로 봄)
테스트
- createAdminSchema: 대문자 아이디 허용, 허용 외 문자(@) 거절 케이스 추가
- createSellerSchema: 대문자 거절 케이스를 허용 외 문자로 교체, 대문자 아이디를 입력 그대로 보내는 케이스 추가
- admins 페이지: 바뀐 오류 문구로 맞춤
BE 릴리즈 #527(main b3fe44f) 기준 schema:pull → codegen. - 생성 타입 변경: AdminCreateAdminInput·AdminCreateSellerInput.username 설명(영문 대소문자 허용)만 - 스냅샷에는 그동안 반영 안 된 판매자 앱용 SDL 추가분도 함께 들어옴(관리자 FE 문서가 쓰지 않아 생성물 영향 없음)
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
요약
[a-z0-9._-]{4,80}→[A-Za-z0-9._-]{4,80}(create-admin-schema·create-seller-schema)Ops.Admin이 있으면ops.admin은 기존USERNAME_TAKEN처리로 막힙니다.b3fe44f) 기준으로 동기화했습니다(별도 커밋).username설명만 바뀌었습니다.테스트
createAdminSchema: 대문자 아이디 허용, 허용 외 문자(@) 거절 케이스를 추가했습니다.createSellerSchema: 대문자 거절 케이스를 허용 외 문자 거절로 바꾸고, 대문자 아이디를 입력 그대로 보내는 케이스를 추가했습니다.admins페이지 spec의 오류 문구를 맞췄습니다.pnpm validate(pre-push)를 통과했습니다.플랜 대조
[A-Za-z0-9._-]{4,80}schema:pull(main, BE 릴리즈 뒤) →codegen, 바뀐 부분 커밋Lumi케이스 이동admins.spec·sellers.spec문구 맞춤sellers.spec은 아이디 오류 문구를 검사하지 않아 바꿀 곳이 없었습니다.pnpm validate→ PR → 머지 게이트 → develop 머지