Conversation
* chore: host_university 컬럼 제약 조건 수정 * refactor: 저장 시에 대한 조건 수정
* feat: 관리자 대학 이미지 업로드를 생성 수정에 통합 - 관리자 대학 생성/수정 API에서 multipart로 대학 정보와 이미지 파일을 함께 받도록 변경 - 이미지 URL을 요청 DTO에서 제거하고 서버에서 S3 업로드 결과를 저장하도록 변경 - 영문명 slug와 한글명 hash를 조합해 중복 영문명 간 S3 경로 충돌을 방지 - 업로드 또는 DB 저장 실패 시 새로 업로드된 이미지가 남지 않도록 보상 삭제 처리 * feat: 관리자 대학 이미지 업로드 테스트 보강 - 대학 생성/수정 시 이미지 업로드 URL 저장과 기존 이미지 유지 동작 검증 - 한글명 기반 S3 경로 식별자와 중복 영문명 충돌 방지 검증 - 업로드 실패 보상 삭제와 삭제용 key 사용 여부 검증
sukangpunch
requested review from
Gyuhyeok99,
Hexeong,
JAEHEE25,
lsy1307,
whqtker and
wibaek
as code owners
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f14b384685
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| private String englishName; | ||
|
|
||
| @Column(name = "format_name", nullable = false, length = 100) | ||
| @Column(name = "format_name", nullable = false, length = 200) |
There was a problem hiding this comment.
This raises host_university.formatName to 200 characters, but the admin POST/PUT request DTOs still declare @Size(max = 100) on formatName (AdminHostUniversityCreateRequest and AdminHostUniversityUpdateRequest). In the /admin/host-universities create/update path, Bean Validation rejects 101-200 character values before the service runs, so the new schema capacity is unusable through these admin APIs even though imports were updated to accept 200.
Useful? React with 👍 / 👎.
4 participants
No description provided.