Skip to content

feat: Select 앨범 시 이모지 로딩 시간 통일#202

Merged
dasosann merged 5 commits intomainfrom
fix/select-album-modal-time
Mar 30, 2026
Merged

feat: Select 앨범 시 이모지 로딩 시간 통일#202
dasosann merged 5 commits intomainfrom
fix/select-album-modal-time

Conversation

@dasosann
Copy link
Copy Markdown
Contributor

요약

구현 사항

  • 앨범 걸렸을때 선택페이지 업로드 시 이모지모달 로딩 시간 통일
  • [ ]
  • [ ]
  • [ ]

📸 스크린샷

Need Review

  • ~ 부분 이렇게 구현했어요, 피드백 부탁해요!

Reference

📜 리뷰 규칙

Reviewer는 아래 P5 Rule을 참고하여 리뷰를 진행합니다.
P5 Rule을 통해 Reviewer는 Reviewee에게 리뷰의 의도를 보다 정확히 전달할 수 있습니다.

  • P1: 꼭 반영해주세요 (Comment)
  • P2: 적극적으로 고려해주세요 (Comment)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
fe Ready Ready Preview, Comment Mar 30, 2026 6:09am

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces an artificial delay after image uploads in SelectAlbumBody.tsx to allow for backend processing, utilizing a try...finally block to manage the uploading state. The review feedback highlights a potential concurrency issue where the upload button should be disabled using the mutation's isLoading state to prevent multiple submissions. Additionally, it is recommended to refactor the duplicated constants and wait-time calculation logic into a shared utility file to improve maintainability.

@dasosann dasosann merged commit 11855df into main Mar 30, 2026
4 checks passed
@dasosann dasosann deleted the fix/select-album-modal-time branch March 30, 2026 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant