Skip to content
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

Merged
merged 5 commits into from
Feb 8, 2025

Conversation

WooGi1020
Copy link
Collaborator

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
Copy link

github-actions bot commented Feb 8, 2025

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

Copy link
Collaborator

@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 사용해서 같이 쓰면 되는거죠..?!!

@@ -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'}
Copy link
Collaborator

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)}
Copy link
Collaborator

Choose a reason for hiding this comment

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

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

Copy link
Collaborator

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: 에러 페이지 적용
2 participants