Conversation
Walkthrough이 변경사항은 파일 업로드 최대 용량을 500KB에서 1MB로 상향하고, 이에 맞춰 UI 안내문구와 파일 유효성 검사 로직을 수정했습니다. 또한, 로그인 모달의 폰트 및 레이아웃이 반응형으로 조정되었고, Tailwind CSS에 새로운 폰트 크기가 추가되었습니다. 파비콘 및 페이지 타이틀도 변경되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant FileUploadUI
participant useImageUpload
User->>FileUploadUI: 파일 선택
FileUploadUI->>useImageUpload: validateFile(file)
useImageUpload->>FileUploadUI: 파일 크기 1MB 이하 여부 반환
FileUploadUI->>User: (1MB 초과 시) 알림 표시
Possibly related PRs
Poem
✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
src/shared/hooks/useImageUpload.ts (1)
27-31: 제한 용량 단위 일관성 제안
현재1000 * 1024바이트를 1MB로 간주하고 있으나, 1MB는 명확히1024 * 1024바이트입니다. 상수로 분리하여 사용하거나 계산식을1024 * 1024로 변경해 보세요.- if (file.size > 1000 * 1024) { - alert('파일 크기는 1MB를 초과할 수 없습니다.'); + const MAX_FILE_SIZE = 1024 * 1024; // 1MB + if (file.size > MAX_FILE_SIZE) { + alert('파일 크기는 1MB를 초과할 수 없습니다.');src/features/event/ui/FileUpload.tsx (1)
33-33: UI 안내문구 확장자 명시 제안
현재 메시지에jpeg, png만 표기되어 있지만,.jpg확장자도 업로드 가능합니다. 사용자 혼동을 줄이기 위해jpg, jpeg, png로 안내 문구를 확장하는 것을 고려해보세요.- <h2 className="text-placeholderText text-xs md:text-sm">1MB 이하의 jpeg, png 파일만 등록할 수 있습니다.</h2> + <h2 className="text-placeholderText text-xs md:text-sm">1MB 이하의 jpg, jpeg, png 파일만 등록할 수 있습니다.</h2>src/widgets/main/ui/LoginModal.tsx (1)
15-17: 함수명 오타 수정 제안
gooleLogin→googleLogin으로 수정하면 가독성이 높아집니다.- const gooleLogin = () => { + const googleLogin = () => { window.location.href = `${import.meta.env.VITE_API_BASE_URL}/oauth2/authorization/google`; }; ... - onClick={gooleLogin} + onClick={googleLogin}
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
public/assets/MainLogo.svgis excluded by!**/*.svg
📒 Files selected for processing (5)
index.html(1 hunks)src/features/event/ui/FileUpload.tsx(1 hunks)src/shared/hooks/useImageUpload.ts(1 hunks)src/widgets/main/ui/LoginModal.tsx(1 hunks)tailwind.config.js(1 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (1)
src/widgets/main/ui/LoginModal.tsx (1)
design-system/ui/buttons/HorizontalCardButton.tsx (1)
HorizontalCardButton(12-40)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: storybook
🔇 Additional comments (4)
src/shared/hooks/useImageUpload.ts (1)
28-30: 파일 크기 제한 상향 및 안내 문구 일치 확인
변경된 파일 크기 제한(1MB)과 알림 메시지가 일치합니다.tailwind.config.js (1)
54-60: 새로운 폰트 크기 확장 확인
text-19,text-20,text-21,text-23,text-24클래스가 tailwind 빌드 후 정상적으로 생성되는지 확인해주세요.src/widgets/main/ui/LoginModal.tsx (1)
29-30: 반응형 폰트 크기 적용 확인
text-21 md:text-26및text-18 md:text-21이 Tailwind 설정과 일치합니다. 모바일(sm)/데스크탑(md) 뷰에서 예상대로 적용되는지 확인해 보세요.Also applies to: 32-32
index.html (1)
5-7:Details
✅ Verification successful
파비콘 및 타이틀 존재 여부 확인
/assets/MainLogo.svg가public/assets에 있는지,<title>이 의도한 대로 출력되는지 검증해주세요.
🏁 Script executed:
#!/bin/bash # public/assets/MainLogo.svg 존재 확인 if [ ! -f public/assets/MainLogo.svg ]; then echo "favicon 파일이 존재하지 않습니다!" exit 1 else echo "favicon 파일 확인 완료." fiLength of output: 88
아래 스크립트로
index.html의<title>이 의도대로 설정됐는지 확인해주세요.
🏁 Script executed:
#!/bin/bash # index.html에서 title 태그 확인 if grep -q '<title>같이가요</title>' index.html; then echo "title 확인 완료: $(grep -Eo '<title>.*</title>' index.html)" else echo "title이 의도한 대로 설정되어 있지 않습니다!" exit 1 fiLength of output: 160
파비콘 및 타이틀 확인 완료
public/assets/MainLogo.svg가 정상적으로 존재합니다.index.html5–7행에<title>같이가요</title>이 의도대로 설정되어 있습니다.
2 participants
Summary by CodeRabbit
신규 기능
스타일
기타