-
Notifications
You must be signed in to change notification settings - Fork 6
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: (어드민) 게시물 생성 페이지 및 기능 구현 #276
Feat: (어드민) 게시물 생성 페이지 및 기능 구현 #276
Conversation
@ParkSohyunee is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel. A member of the Team first needs to authorize it. |
{item.type === 'notice' && <p className={styles.articleNotice}>{item.description}</p>} | ||
{item.type === 'note' && <textarea value={item.description} readOnly className={styles.articleNotice} />} |
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.
@Nahyun-Kang 나현님, 공지 조회 컴포넌트에서 유의사항 카테고리가 'note'로 표기됨에 따라 관련 타입을 수정 및 textarea로 보여주도록 수정한 점 참고 부탁드립니다. 👀
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.
오오, textarea를 이렇게도 사용할 수 있군요! 줄바꿈을 위해 p블록 대신 textarea readOnly 사용!! 배워갑니다!🤔
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.
@Nahyun-Kang 나현님, 공지 조회 컴포넌트에서 유의사항 카테고리가 'note'로 표기됨에 따라 관련 타입을 수정 및 textarea로 보여주도록 수정한 점 참고 부탁드립니다. 👀
소현님! note로 타입 변경된 점 확인했습니다! 노티 감사합니다!!🙇♀️
export interface NoticeContentType { | ||
type: string; | ||
description: string; | ||
imageUrl: string; | ||
buttonName: string; | ||
buttonLink: string; | ||
[key: string]: unknown; | ||
type: NoticeContentsType; | ||
description?: string; | ||
imageUrl?: unknown; // FileList 타입일수도, String 타입일수도, undefined일수도 있으므로 사용처에서 타입 지정 | ||
buttonName?: string; | ||
buttonLink?: string; | ||
} |
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.
@Nahyun-Kang 나현님, 공지 콘텐츠 타입을 공지 생성과 조회 모두에서 사용될 수 있도록 수정했습니다. 더불어, 카테고리 타입도 함께 작성해 두어 참고 부탁드립니당 👀
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.
타입 조회에서도 공통으로 쓸 수 있게 작성해주셔서 감사합니다! NoticeContentType으로 불러와서 쓸 수 있도록 하겠습니당!👍👍
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.
LGGGGTM!! 소현님, 컴포넌트 수가 어마어마 하네요🥹
아예 새로운 어드민 파트에서, 또 복잡한 게시물 생성을 맡아주셔서 감사합니다...!
기획하면서도 이게 맞나 싶었는데, 기획안을 그대로 반영해주셔서 정말 감사해요🙇♀️✨
{item.type === 'notice' && <p className={styles.articleNotice}>{item.description}</p>} | ||
{item.type === 'note' && <textarea value={item.description} readOnly className={styles.articleNotice} />} |
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.
오오, textarea를 이렇게도 사용할 수 있군요! 줄바꿈을 위해 p블록 대신 textarea readOnly 사용!! 배워갑니다!🤔
와 소현님 공지 생성 부분을 이렇게 만들어주시다니.. 너무 멋져요..🥹 남겨주신 코멘트도 확인해보았습니다! 너무 LGTM! |
개요
작업 사항
참고 사항
마크다운 라이브러리 선택 이유
마크다운 라이브러리로 다음과 같이 3개의 라이브러리를 검토했습니다. 그 중 가볍고, 현재도 꾸준히 업데이트 되고있고, next.js와 호환이 잘되고, 필요로한 기능 추가 플러그인 설치 없이 사용할 수 있는 @uiw/react-md-editor를 선택했습니다.
후속 작업
관련 이슈
스크린샷
리뷰어에게
긴 PR 읽어 주셔서 감사합니다. 💕