-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat: 시연 때 발생한 오류 및 에러 페이지 적용 #133
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다! errorBoundary
도 fallback
사용해서 같이 쓰면 되는거죠..?!!
@@ -90,7 +90,7 @@ const MatchingInfoItem = ({ | |||
<div className="flex-1 overflow-y-scroll scroll-hidden overflow-hidden mt-5"> | |||
<p className="font-medium text-body"> | |||
<span>추가 멘트 : </span> | |||
{manualInfo.description} | |||
{manualInfo.description ? manualInfo.description : 'X'} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 없을때 이렇게 표현해주는게 훨씬 나을듯 하네요! 👍🏻👍🏻
@@ -7,7 +14,7 @@ const Tags = ({ tags }: { tags: string[] }) => { | |||
key={tag} | |||
className="text-assistive min-w-fit text-black font-medium px-3 py-1 rounded-full bg-primary" | |||
> | |||
# {tag} | |||
# {showLabel(tag)} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
엇 영어로 보이던 tag들 수정됐군요! 감사합니당 🙇🏻♀️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
흐억 이 부분 마지막 날에 급하게 하느라고... 코드가 좀 이상했는데 수정해주셨네요 감사합니닷 ...!!
1. 무슨 이유로 코드를 변경했나요?
2. 어떤 위험이나 장애를 발견했나요?
3. 관련 스크린샷을 첨부해주세요.
4. 완료 사항
시연떄 발생헀던 자잘한 오류들 (태그 데이터형태 그대로 출력, 블랙리스트 이미지 누락 등) 수정하고 만들어두신 에러 페이지
react-error-boundary
라이브러리 사용해서 적용했습니다~5. 추가 사항