feat(community): 공지사항 상세글 페이지 #45
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
| function Content({ content }: { content: string }) { | ||
| return <div className={styles.content}>{content}</div>; | ||
| } |
There was a problem hiding this comment.
Content의 content는 Render Props 패턴으로 되어 있는데 의도하신 이유가 있나요? children이 더 적절해보이는데 특별한 이유가 있는지 궁금합니다.
There was a problem hiding this comment.
현재 목데이터로 swagger의 예시 데이터를 사용하고 있는데요, 저도 children이 더 적절하다고 생각하였으나 해당 목데이터의 content가 노드가 아닌 단순 string 값으로 설정되어 있어 render props로 렌더링 중입니다! 혹시 적절한 방법이 있을까요?
일관적인 디자인을 가져가는게 좋다고 생각해요. 일단은. 나중에 다르게 해야할 특별한 요구사항이 있다면 그때 변경하는것으로 해요. 추가로 기존에는 다음글 이전글과 같이 하단에 네비게이터도 있는데요. 해당 페이지는 너무 심플하다고 생각해요. 다른 레퍼런스도 찾아보면서 조금 완성도를 높여보는 건 어떨까요? 
|
적극 동의합니다! 현재 상태에서 추가할 수 있는 기능을 고려해볼게요😊 |
… into feature/notice-detail-page
"목록으로 돌아가기" 버튼을 추가하였어요! |
|
@SunwooJaeho 이전글이나 다음글 등 다른 포스트 네이게이터를 추가할 계획은 없나요? |
현재 api 상에서는 이전글과 다음글에 대한 정보를 불러오지 않아 어떤 방식으로 구현하면 좋을 지 고민 중입니다! 따라서 현재는 목록으로 돌아가는 버튼만 추가해두었어요😄 |
Summary
공지사항 상세글 페이지 퍼블리싱을 했어요.
Tasks
To Reviewer
모든 상세글 페이지의 디자인을 해당 레이아웃과 동일하게 해도 좋을 지에 대한 의견이 궁금합니다!
(최소한 학부 소식은 동일 레이아웃으로 가는 걸 고민 중입니다!)
Screenshot