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.

Sign up for GitHub

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

Jump to bottom

[#486] [책 상세 / 모임 상세] 코멘트 목록 컴포넌트 #487

Merged
merged 13 commits into from
Feb 20, 2024
Merged

[#486] [책 상세 / 모임 상세] 코멘트 목록 컴포넌트 #487

merged 13 commits into from
Feb 20, 2024

Conversation

gxxrxn
Copy link
Member

@gxxrxn gxxrxn commented Feb 6, 2024 •
edited
Loading

구현 내용

  • 코멘트 작성자에 대한 타입을 Writer로 정의했어요.
  • useBookCommentsQuery에 suspense를 적용했어요.
  • CommentList 컴포넌트를 구현했어요.
    • 구현된 컴포넌트는 스토리북에서 볼 수 있어요!
    • 모임 상세 페이지, 책 상세 페이지에서 공통적으로 사용되는 코멘트 목록 UI를 표현해요. (presentational)
  • BookGroupCommentList, BookCommentList 컴포넌트를 구현했어요.
    • 모임 코멘트, 책 코멘트 api를 호출하여 CommentList 컴포넌트에 데이터를 전달해요. (container)

스크린샷

pr 포인트

Help

  • 개선하면 좋을 부분(ex. 변수명, 패턴 등..)이 있다면, 꼭꼭 말씀해주세요!! 🙇🏻‍♀️

관련 이슈

@gxxrxn gxxrxn added 🐥 프론트 필수! ✨ feature New feature or request 🔥 v.1.0 New feature for releasing v.1.0 labels Feb 6, 2024
@gxxrxn gxxrxn self-assigned this Feb 6, 2024
@vercel Vercel
Copy link

vercel bot commented Feb 6, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dadok ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 19, 2024 4:03pm

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 6, 2024 •
edited
Loading

🚀 Storybook: https://64aba10cd478ad51307a8997-lqvcaajlqw.chromatic.com/

@gxxrxn gxxrxn marked this pull request as ready for review February 11, 2024 23:40
hanyugeon
hanyugeon reviewed Feb 13, 2024
View reviewed changes
Copy link
Member

@hanyugeon hanyugeon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

민종님이 사용하셨던 presentational container 패턴을 잘 활용해주신것 같네요 👍
몇 가지 코멘트 남겨두었으니 확인해주시고 편하게 의견 남겨주세요

src/v1/comment/CommentList.tsx Outdated Show resolved Hide resolved
src/v1/comment/CommentList.tsx Outdated
Comment on lines 33 to 34
hidden,
hiddenText,
Copy link
Member

Choose a reason for hiding this comment

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

p3;

hidden과 hiddenText props는 독서 모임 댓글에만 사용되므로 group처럼 모임과 관련된 키워드가 포함되면 좋을것같다는 생각이 들었어요.

hidden또한 bool 타입임을 암묵적으로 알리기 위한 is 접두사가 있다면 좋을 것 같아요...!

Copy link
Member Author

@gxxrxn gxxrxn Feb 19, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

hidden과 hiddenText props는 독서 모임 댓글에만 사용되므로 group처럼 모임과 관련된 키워드가 포함되면 좋을것같다는 생각이 들었어요.

컴포넌트의 재사용성을 높이기 위해 presentational 컴포넌트를 작성한만큼 도메인과 관련된 키워드를 prop에 포함하는건 지양하고 싶었어요.. 그리고 댓글을 숨기는 기능은 독서모임 뿐만 아니라 코멘트 목록을 보여줄 때 충분히 재사용될 수 있을 것 같아서 CommentList에 포함시켰는데, 특정한 경우에만 사용되는 기능이라면 오히려 presentational 컴포넌트에서 제거하는 방향이 좋다고 생각해요!

hidden또한 bool 타입임을 암묵적으로 알리기 위한 is 접두사가 있다면 좋을 것 같아요...!

좋아요!! 👍🏻

src/v1/comment/CommentList.tsx Outdated Show resolved Hide resolved
src/v1/comment/CommentList.tsx Outdated Show resolved Hide resolved
src/v1/comment/CommentList.tsx Outdated Show resolved Hide resolved
src/v1/comment/CommentList.tsx Outdated
Comment on lines 40 to 43
const titleOnCommentChange = useMemo(
() => [name, '수정하기'].join(' '),
[name]
);
Copy link
Member

Choose a reason for hiding this comment

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

p4;

name 파라미터를 통해 drawer의 title을 반환하는 함수이므로
getChangeDrawerTitle? 이런 네이밍은 어떤가요!?

Copy link
Member Author

Choose a reason for hiding this comment

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

useMemo를 사용했기 때문에 함수가 아니라 값을 저장하고 있는 변수라고 이해하는게 맞지 않을까요? 코멘트 수정 시에 사용되는 title 이라는 의미를 담고 싶었어요.🥲

minjongbaek
minjongbaek previously approved these changes Feb 14, 2024
View reviewed changes
Copy link
Member

@minjongbaek minjongbaek left a comment

Choose a reason for hiding this comment

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

권한과 엮여있는 로직을 깔끔하게 잘 해결하신 것 같아요. 👍
재현님께서 남겨주신 코멘트만 확인해주시면 될 것 같아요~

minjongbaek
minjongbaek approved these changes Feb 20, 2024
View reviewed changes
Copy link
Member

@minjongbaek minjongbaek left a comment

Choose a reason for hiding this comment

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

👍

hanyugeon
hanyugeon approved these changes Feb 20, 2024
View reviewed changes
Copy link
Member

@hanyugeon hanyugeon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!! 👍

@gxxrxn gxxrxn merged commit 9267ba6 into main Feb 20, 2024
3 checks passed
@minjongbaek minjongbaek deleted the feat/#486 branch March 4, 2024 16:07
gxxrxn added a commit that referenced this pull request Jun 17, 2024
@gxxrxn
[#486] [책 상세 / 모임 상세] 코멘트 목록 컴포넌트 (#487)
f452642 
* refactor: writer 타입 정의

* feat: CommentList 컴포넌트 구현 (presentaional)

* feat: 책 코멘트 query에 suspense 적용

- useBookComments 쿼리 추가

* feat: BookGroupCommentList 구현 (contaier)

* feat: BookCommentList 구현 (container)

* feat: CommentList 컴포넌트 스토리북 작성

* chore: 사용되지 않는 이전 CommentList 컴포넌트 삭제

* refactor: book comments query key 추가

* refactor: 코멘트 메뉴 관련 props 수정

* feat: 코멘트목록 name, 이벤트 핸들러 props 추가

* refactor: 변수, prop 이름 수정

* fix: storybook props 수정
gxxrxn added a commit that referenced this pull request Aug 20, 2024
@gxxrxn
Revert "[#486] [책 상세 / 모임 상세] 코멘트 목록 컴포넌트 (#487)"
6ca5a19 
This reverts commit f452642.
gxxrxn added a commit that referenced this pull request Aug 20, 2024
@gxxrxn
revert "Release v1.0.0" (#688)
c6c0c84 
* Revert "[#634] v1 디렉토리 네이밍 수정 (#635)"

This reverts commit 86c304a.

* Revert "[#632] 사용하지 않는 라이브러리 제거 (#633)"

This reverts commit 34b4efb.

* Revert "[#627] 불필요한 utils, hooks, constants 정리 및 import문 통일 (#631)"

This reverts commit 254f6bb.

* Revert "[#629] tailwind config color 정리 (#630)"

This reverts commit 595f476.

* Revert "[#622] BottomActionButton 패딩 수정 (#626)"

This reverts commit 9796dab.

* Revert "[#625] Toast 컴포넌트 icon이 잘리는 문제 해결 (#628)"

This reverts commit e868efc.

* Revert "[#623] [도서 검색] Input 검색 아이콘 Layout Shift 해결 (#624)"

This reverts commit d0392d6.

* Revert "[#619] PWA 구현 (#621)"

This reverts commit 587f990.

* Revert "[#616] 책장 좋아요 뱃지 UI 수정 (#620)"

This reverts commit 70af27a.

* Revert "[#349] 메타 데이터 추가 (#617)"

This reverts commit 209ba3e.

* Revert "[#556] 버튼 컴포넌트 disabled style 작성 (#615)"

This reverts commit 0425223.

* Revert "[#610] 로그인 페이지 구현 (#614)"

This reverts commit badcf8e.

* Revert "[#608] 페이지 접근 권한 부여 (#611)"

This reverts commit 7f1cd21.

* Revert "[#612] 북카이브 페이지에서 발생하는 무한 리랜더링 버그 수정 (#613)"

This reverts commit bac4ff2.

* Revert "[#605] Google LightHouse CI 추가 (#606)"

This reverts commit d6cccdb.

* Revert "[#602] [모임 생성] 퍼널 progress bar, stepper 구현 (#607)"

This reverts commit bcbdc67.

* Revert "[#603] [내 프로필] ErrorBoundary 제거 (#604)"

This reverts commit f044cd2.

* Revert "feat: start-ssl script 작성 (#601)"

This reverts commit 8de7079.

* Revert "[#596] accessToken이 만료된 직후 다른 페이지 접근할 때 에러 페이지를 보여주지 않고 새로고침 (#600)"

This reverts commit 59407c6.

* Revert "[#592] [도서 검색] 검색 키워드 및 결과 유지 기능 구현 (#593)"

This reverts commit af2413c.

* Revert "[#595] Toast UI 개선 (#597)"

This reverts commit 52e9992.

* Revert "[#591] 3D 책 컴포넌트, 책장 페이지 개선 (#594)"

This reverts commit 2beb0f2.

* Revert "[#570] [도서 검색] 도서 검색 결과 렌더링 시 search 헤더 숨기고 input sticky 적용 (#589)"

This reverts commit 0ff4ca0.

* Revert "[#588] 폰트 디자인 시스템 적용 (#590)"

This reverts commit 6a054a4.

* Revert "[#557] [도서 검색] 최근 검색어 api 요청 (#587)"

This reverts commit 591e9a8.

* Revert "[#585] [프로필 페이지] '참여한 모임' 영역 사용자가 모임장인 모임에 마크 추가 (#586)"

This reverts commit 3a3f263.

* Revert "[#575] Drawer 컴포넌트 개선 (#576)"

This reverts commit cff01fd.

* Revert "[#581] [모임 생성] 모임 생성 퍼널 페이지 작성 (#582)"

This reverts commit 55a08ee.

* Revert "[#583] Switch 컴포넌트 style 수정 (#584)"

This reverts commit e7798f7.

* Revert "[#578] iOS 환경에서의 DatePicker UI 수정 (#579)"

This reverts commit 24b42bf.

* Revert "[#574] FloatingButton 버그 수정 (#577)"

This reverts commit dd37ff5.

* Revert "[#559] BookInfoCard 컴포넌트 수정 및 goToBookSelectStep 이벤트 추가 (#572)"

This reverts commit f1c6a3c.

* Revert "[#571] Drawer 컴포넌트 style 개선 (#573)"

This reverts commit f21ada4.

* Revert "[#561] [모임 생성] 모임 상세 퍼널 개선 (#567)"

This reverts commit bbd1123.

* Revert "[#552] next custom server 구동 시 hmr이 동작하지 않는 문제 해결 (#569)"

This reverts commit 019abd5.

* Revert "[#563] [모임 생성] 모임 가입문제 퍼널 (#568)"

This reverts commit 4d6a14f.

* Revert "[#564] BottomActionButton 컴포넌트 position 속성 수정 (#566)"

This reverts commit 94365ee.

* Revert "[#562] [모임 생성] 모임 이름 퍼널 (#565)"

This reverts commit 0bd1349.

* Revert "[#543] [모임 생성] 책 선택 퍼널 (#560)"

This reverts commit b2f5236.

* Revert "[#553] [모임 생성] 모임 상세 퍼널 (#558)"

This reverts commit 31a6d84.

* Revert "[#554] 스토리북 preview decorator에서 Layout 제거 (#555)"

This reverts commit 495321e.

* Revert "[#549] 'unable to verify first certificate' 에러 해결 (#550)"

This reverts commit bdfa458.

* Revert "fix: 모바일 환경에서의 스크롤 버그 수정 (#551)"

This reverts commit 1d56794.

* Revert "[#546] [독서모임] 모임 페이지 누락된 작업 및 버그 수정 (#547)"

This reverts commit 448e8b5.

* Revert "[#531] 로컬 서버에 https 적용 (#541)"

This reverts commit f2f4a82.

* Revert "[#540] [모임 생성] 모임 상세 퍼널 마크업 (#542)"

This reverts commit 5d23ab3.

* Revert "[#544] vscode react component snippet 작성 (#545)"

This reverts commit 68dc181.

* Revert "[#535] [독서모임] 독서모임 목록 페이지 개선 (#539)"

This reverts commit 56137df.

* Revert "[#505] [모임 상세] 모임 삭제 기능 구현 (#537)"

This reverts commit efdb2c5.

* Revert "[#530] FloatingButton 컴포넌트 적용 (#536)"

This reverts commit 3671624.

* Revert "[#529] redirect 페이지 개선, 토큰 업데이트 로직 개선 (#532)"

This reverts commit 0867ee4.

* Revert "[#523] [모임 상세] 모임 수정 페이지 (#527)"

This reverts commit f889a79.

* Revert "[#526] 도서 검색 페이지 개선 (#528)"

This reverts commit 81b39b7.

* Revert "[#524] Menu 컴포넌트 개선 (#525)"

This reverts commit 52687da.

* Revert "[#521] TextArea 컴포넌트 (#522)"

This reverts commit a1673f6.

* Revert "[#518] Input 컴포넌트 style 추가 (#519)"

This reverts commit 2bd859a.

* Revert "[#516] DatePicker 컴포넌트 (#520)"

This reverts commit 63af14d.

* Revert "[#504] [모임 상세] 모임 게시글 작성 기능 구현 (#517)"

This reverts commit e6f8d87.

* Revert "[#509] [레이아웃] BottomNavigation 버그 및 UI 수정 (#510)"

This reverts commit ada909a.

* Revert "feat: 모임 게시글 수정, 삭제 기능 구현 (#513)"

This reverts commit 36c1e71.

* Revert "[#507] [프로필] 로그아웃 햄버거 구현 (#508)"

This reverts commit 90c5855.

* Revert "feat: hosts 파일을 수정할 수 있는 updateDevHost script 작성 (#511)"

This reverts commit cd297e9.

* Revert "[#497] [책 상세] 책 상세 페이지 api 연결 (#500)"

This reverts commit 7842028.

* Revert "[#498] useFunnel 작성 (#501)"

This reverts commit e6f7878.

* Revert "[#489] [도서 검색] 도서검색 페이지 개선 작업 (#490)"

This reverts commit 0043fe7.

* Revert "[#491] [도서 검색] 베스트셀러 클릭 시 라우팅 경로 수정 (알라딘 -> 다독다독) (#492)"

This reverts commit e68d1d3.

* Revert "[#493] 에러 페이지 (#494)"

This reverts commit e95c940.

* Revert "[#495] [책 상세] 책 상세 페이지 마크업 (#496)"

This reverts commit 052e6d4.

* Revert "[#474] 리프레시 토큰이 만료된 경우 에러 페이지로 넘어가는 이슈 수정 (#483)"

This reverts commit 633da9f.

* Revert "[#486] [책 상세 / 모임 상세] 코멘트 목록 컴포넌트 (#487)"

This reverts commit f452642.

* Revert "[#443] TopNavigation 컴포넌트 스타일 fixed 적용 (#488)"

This reverts commit 60d3311.

* Revert "[#468] [검색] 검색 페이지 작성 (#478)"

This reverts commit 313d61a.

* Revert "[#481] [책 상세] 책 정보 컴포넌트 (#482)"

This reverts commit 03fa16e.

* Revert "[#442] 스켈레톤 컴포넌트 작성 (#480)"

This reverts commit 22b06a2.

* Revert "[#453] [모임] 모임 목록 페이지 개선 (#475)"

This reverts commit c6b1ab1.

* Revert "typo: 오타 수정 (#485)"

This reverts commit 395e050.

* Revert "[#476] [모임 상세] 비로그인 시 모임 참여하기 버튼 비활성화 (#479)"

This reverts commit a2e628d.

* Revert "[#472] Drawer 컴포넌트 (#477)"

This reverts commit 5e54b72.

* Revert "[#470] Menu 컴포넌트 (#471)"

This reverts commit 0a4705d.

* Revert "[#465] [모임] BookInfoCard 컴포넌트 추상화 (#466)"

This reverts commit 81bc6d8.

* Revert "[#467] 내가 가입한 모임 상세 페이지 (#469)"

This reverts commit 75070ce.

* Revert "[#463] Loading 컴포넌트 animation 버그 해결 (#464)"

This reverts commit 8219f67.

* Revert "[#456] [프로필] 프로필 페이지 (#461)"

This reverts commit 150592c.

* Revert "[#455] [모임 상세] 모임 가입 문제 페이지 (#462)"

This reverts commit 7bbdbaf.

* Revert "[#459] [유저 로그인] 로그인 BottomSheet 컴포넌트 작성 (#460)"

This reverts commit da3fb33.

* Revert "[#457] LikeButton 컴포넌트 (#458)"

This reverts commit 9d1a5c7.

* Revert "베이스 컴포넌트 위치 변경 (#454)"

This reverts commit fd8ad31.

* Revert "[#447] [모임 상세] 모임 참여 여부, 모임장 여부에 따른 UI 구현 (#450)"

This reverts commit 61ada97.

* Revert "[#449] [책장] 책장 상세 페이지 (#451)"

This reverts commit 1f389e4.

* Revert "[#445] Layout 컴포넌트에서 TopHeader 컴포넌트 분리 (#452)"

This reverts commit 9d11c70.

* Revert "[#415]  [프로필] 프로필 생성 페이지 (#427)"

This reverts commit 4175612.

* Revert "[fix] group 페이지 타입 에러 수정 (#448)"

This reverts commit 66005ea.

* Revert "[#434] [모임 상세] 독서모임 상세 페이지 api 연결 (#441)"

This reverts commit 617614f.

* Revert "[#439] [독서모임] 모임 목록 페이지 리팩토링 (#440)"

This reverts commit f50f393.

* Revert "[#414] [프로필] 프로필 수정 페이지 (#426)"

This reverts commit 91b8171.

* Revert "[#437] [독서모임] Avartar, bookGroupStatus 적용 및 typo 수정 (#438)"

This reverts commit b56d3d5.

* Revert "[#412] [북카이브] 북카이브 페이지 (회원) (#436)"

This reverts commit 434dd73.

* Revert "[refactor] bookcover v1 폴더로 이동 (#433)"

This reverts commit 434dd8c.

* Revert "[refactor] 폴더 및 파일 구조, 네이밍 수정 (#432)"

This reverts commit 5ba2e21.

* Revert "[fix] github 에서 대소문자 구문하지 못하는 문제 해결 (#431)"

This reverts commit 549a5e9.

* Revert "[#403] [모임 상세] 모임 정보 컴포넌트 (#423)"

This reverts commit 378828c.

* Revert "fix: 이벤트 핸들러 명칭 변경"

This reverts commit de6588c.

* Revert "fix: 컴포넌트명 변경"

This reverts commit 03899d7.

* Revert "fix: 스타일 관련 코드 수정 및 불필요한 요소 제거"

This reverts commit 14a6b84.

* Revert "feat: 내가 가입한 모임 simple ui 구현"

This reverts commit 5dd3e60.

* Revert "[#410] [북카이브] 북카이브 페이지 (비회원) (#425)"

This reverts commit 08c919d.

* Revert "[#411][독서 모임] Detail 독서 모임 컴포넌트 (#428)"

This reverts commit cbb032a.

* Revert "[#408] [모임 상세] 게시글 컴포넌트 (#429)"

This reverts commit 2277b0d.

* Revert "[#404][독서모임] 모임 검색 컴포넌트 (#419)"

This reverts commit 281beee.

* Revert "[#418] Avatar 컴포넌트 (#420)"

This reverts commit 51d471e.

* Revert "[#406] [모임 상세] 멤버 목록 컴포넌트 (#424)"

This reverts commit 199a234.

* Revert "[#399] Toast 컴포넌트 (#402)"

This reverts commit 9333f87.

* Revert "[#405] [레이아웃] 공통 레이아웃 (#421)"

This reverts commit fb8d8d6.

* Revert "[#398] RadioButton 컴포넌트 (#400)"

This reverts commit 9c8bdc2.

* Revert "[#413] Issue 템플릿에 v1.0 라벨 추가 (#417)"

This reverts commit facb746.

* Revert "[#363] BottomActionButton 컴포넌트 (#395)"

This reverts commit a653d3b.

* Revert "[#366] BottomSheet 컴포넌트 (#394)"

This reverts commit a43dea4.

* Revert "[#364] Badge 컴포넌트 (#396)"

This reverts commit 86edc76.

* Revert "[#362] TopNavigation 컴포넌트 리팩터링 (#393)"

This reverts commit 55a6b98.

* Revert "[#361] BottomNavigation 컴포넌트 (#392)"

This reverts commit d7d8ca6.

* Revert "[#388] Input, Select 컴포넌트 에러 처리 (#389)"

This reverts commit f5bf5ef.

* Revert "[#390] storybook snippet 작성 (#391)"

This reverts commit 97984a7.

* Revert "[#365] Modal 컴포넌트 (#387)"

This reverts commit 8e931fc.

* Revert "[#367] Switch 컴포넌트 (#386)"

This reverts commit e1b1ae9.

* Revert "[#382] Assets 정리 (#384)"

This reverts commit 182def7.

* Revert "[#383] Select 컴포넌트 (#385)"

This reverts commit 7c189bc.

* Revert "[#357] TopHeader UI 컴포넌트 및 스토리북 작성 (#379)"

This reverts commit 4dfbcc8.

* Revert "[#358] TopNavigation 컴포넌트 (#378)"

This reverts commit 41f8f99.

* Revert "[#360] Input 컴포넌트 작성 (#380)"

This reverts commit e28792d.

* Revert "[#359] Button 컴포넌트 (#381)"

This reverts commit 6b84755.

* Revert "[#376] Storybook 설정 및 GlobalTheme, Font 수정 (#377)"

This reverts commit 3a7a9e6.

* Revert "[#374] Tailwind 클래스를 정렬하기 위한 Prettier 플러그인 추가 (#375)"

This reverts commit e365160.

* Revert "[#372] Theme, GlobalStyle, Font 세팅 (#373)"

This reverts commit 4f7df42.

* Revert "[#370] Storybook & Headless UI 추가 (#371)"

This reverts commit 7ad8aef.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hanyugeon hanyugeon hanyugeon approved these changes

@minjongbaek minjongbaek minjongbaek approved these changes

Assignees

@gxxrxn gxxrxn

Labels
✨ feature New feature or request 🐥 프론트 필수! 🔥 v.1.0 New feature for releasing v.1.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[책 상세 / 모임 상세] 코멘트 목록 컴포넌트
3 participants
@gxxrxn @minjongbaek @hanyugeon