Skip to content

Conversation

@cccwon2
Copy link
Contributor

@cccwon2 cccwon2 commented Dec 9, 2024

  1. S3 이미지 URL 로 시작되는지 확인하는 로직 추가
  2. S3 이미지가 403 에러 발생시 디폴트 이미지 표시
  3. 날짜 검증 유틸리티 파일에 날짜 형식이 아닌 경우 현재 날짜를 반환하는 로직 추가

스크린샷 2024-12-10 001354

@cccwon2 cccwon2 self-assigned this Dec 9, 2024
@cccwon2 cccwon2 requested a review from imtaejunk December 9, 2024 14:05
@cccwon2 cccwon2 added the 🐞 Fix 버그 수정에 대한 커밋 label Dec 9, 2024
@github-actions
Copy link

github-actions bot commented Dec 9, 2024

💄 Storybook: https://673dcf13c9a3a1189a7c6fd4-sdqachdppf.chromatic.com/ # Chromatic에서 생성된 Storybook URL
🕖 Update: 2024년 12월 10일 01시 37분 34초 # 현재 시간

Copy link
Collaborator

@yyezzzy yyezzzy left a comment

Choose a reason for hiding this comment

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

이미지 url이랑 date에러 나는거 수정해주셨군요 ! 감사합니다 ~

Comment on lines +4 to +8
// 유효한 Date 객체인지 확인
if (!(date instanceof Date) || isNaN(date.getTime())) {
return new Date().toLocaleDateString("ko-KR", { year: "numeric", month: "2-digit", day: "2-digit" });
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

Comment on lines +116 to +117
{/* 이미지 인디케이터 - 유효한 이미지가 2개 이상이고 에러가 없을 때만 표시 */}
{imageUrls.filter((url) => isValidS3Url(url)).length > 1 && !imageError && (
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@hongggyelim hongggyelim left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!

@hongggyelim hongggyelim merged commit 4d445a0 into dev Dec 10, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 Fix 버그 수정에 대한 커밋

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants