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

fix(web-domains): about 페이지에서 answers 리스트 중 comment가 없을 경우 렌더링하지 않도록 수정 #112

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

Doeunnkimm
Copy link
Member

🎉 변경 사항

제곧내..

🔗 링크

QA

🙏 여기는 꼭 봐주세요!

@@ -30,7 +30,7 @@ export const AnsweredQuestionsContainer = () => {
<Accordion.Content>
<div css={answerContentCss}>
<Txt typography="heading3">{answer.content}</Txt>
<Txt typography="body2">{answer.commentContent}</Txt>
{answer.commentContent != null && <Txt typography="body2">{answer.commentContent}</Txt>}
Copy link
Member

Choose a reason for hiding this comment

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

null은 string이나 Number처럼 형변환이 안될거같아서 상관없을거 같긴한데,
체크차원에서 여쭤보자면 !==이 아니라 !=은 의도하신 코드이신거죠?

Copy link
Member Author

Choose a reason for hiding this comment

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

넹 의도했어요. nullish한 것만 거르고 싶었습니다~!

@Doeunnkimm Doeunnkimm merged commit 6b34340 into main Aug 21, 2024
3 checks passed
@Doeunnkimm Doeunnkimm deleted the fix/about-answers-list branch August 21, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants