Skip to content

Conversation

@WooGi1020
Copy link
Contributor

1. 무슨 이유로 코드를 변경했나요?


2. 어떤 위험이나 장애를 발견했나요?


3. 관련 스크린샷을 첨부해주세요.


4. 완료 사항

시연떄 발생헀던 자잘한 오류들 (태그 데이터형태 그대로 출력, 블랙리스트 이미지 누락 등) 수정하고 만들어두신 에러 페이지 react-error-boundary 라이브러리 사용해서 적용했습니다~

5. 추가 사항


@WooGi1020 WooGi1020 added feat 신규피쳐 fix fix labels Feb 8, 2025
@WooGi1020 WooGi1020 requested a review from woneeeee February 8, 2025 06:26
@WooGi1020 WooGi1020 self-assigned this Feb 8, 2025
@WooGi1020 WooGi1020 linked an issue Feb 8, 2025 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Feb 8, 2025

구현 기능 확인하기: https://gachtaxi-l26hhew9l-cogis-projects.vercel.app

Copy link
Member

@woneeeee woneeeee left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! errorBoundaryfallback 사용해서 같이 쓰면 되는거죠..?!!

<p className="font-medium text-body">
<span>추가 멘트 : </span>
{manualInfo.description}
{manualInfo.description ? manualInfo.description : 'X'}
Copy link
Member

Choose a reason for hiding this comment

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

오 없을때 이렇게 표현해주는게 훨씬 나을듯 하네요! 👍🏻👍🏻

className="text-assistive min-w-fit text-black font-medium px-3 py-1 rounded-full bg-primary"
>
# {tag}
# {showLabel(tag)}
Copy link
Member

Choose a reason for hiding this comment

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

엇 영어로 보이던 tag들 수정됐군요! 감사합니당 🙇🏻‍♀️

Copy link
Member

Choose a reason for hiding this comment

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

흐억 이 부분 마지막 날에 급하게 하느라고... 코드가 좀 이상했는데 수정해주셨네요 감사합니닷 ...!!

@WooGi1020 WooGi1020 merged commit fcab4d6 into main Feb 8, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 신규피쳐 fix fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 에러 페이지 적용

3 participants