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

피드 상세 뷰에 해당 모임 피드 리스트 추가 #602

Merged
merged 9 commits into from
Dec 9, 2023

Conversation

eunsukimme
Copy link
Member

@eunsukimme eunsukimme commented Dec 2, 2023

🚩 관련 이슈

📋 작업 내용

  • 피드 게시글 상세페이지에서 해당 모임 최신 피드 불러오는 UI 추가
  • 피드 아이템 컴포넌트 props 수정(meetingid) props 추가

📌 PR Point

  • 피드 아이템 컴포넌트에서 좋아요 기능이 meetingId를 필요로하는데 이게 router.query.id 와 커풀링 되어 있어요. 그런데 게시글 상세페이지는 router.query.id가 모임의 id가 아닌 게시글의 id이기 때문에 다르게 처리해줘야 했어요. 그래서 props로 meetingId를 받을 수 있도록 수정했어요.

📸 스크린샷

Screenshot 2023-12-09 at 8 34 33 PM

Copy link

cloudflare-workers-and-pages bot commented Dec 2, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 450e8d8
Status: ✅  Deploy successful!
Preview URL: https://95758e18.sopt-crew-dev-legacy.pages.dev
Branch Preview URL: https://feat--596.sopt-crew-dev-legacy.pages.dev

View logs

@eunsukimme eunsukimme changed the title feat/#596 피드 상세 뷰에 해당 모임 피드 리스트 추가 Dec 2, 2023
@pull-request-size pull-request-size bot added size/L and removed size/M labels Dec 9, 2023
@eunsukimme eunsukimme changed the base branch from feat/##582 to develop December 9, 2023 10:51
@eunsukimme eunsukimme marked this pull request as ready for review December 9, 2023 11:42
@pull-request-size pull-request-size bot removed the size/L label Dec 9, 2023
@eunsukimme eunsukimme changed the base branch from develop to feat/#615 December 9, 2023 12:22
@pull-request-size pull-request-size bot added size/L and removed size/XL labels Dec 9, 2023
Copy link
Member

@100Gyeon 100Gyeon left a comment

Choose a reason for hiding this comment

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

🚀🚀🚀🚀🚀

const handleLikeClick = (postId: number) => (e: React.MouseEvent<HTMLButtonElement>) => {
e.preventDefault();
mutateLike(postId);
// TODO: ampli logging 필요?
Copy link
Member

Choose a reason for hiding this comment

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

오 챙겨줘서 고마워 이거 필요해!!

image

Base automatically changed from feat/#615 to develop December 9, 2023 15:00
@pull-request-size pull-request-size bot added size/XL and removed size/L labels Dec 9, 2023
eunsukimme and others added 9 commits December 10, 2023 00:00
* feat: 댓글 입력 placeholder 변경

* feat: 좋아요 영역 hover 처리

* feat: 댓글 수 표시 영역을 버튼으로 spec 변경, 댓글 버튼 클릭 시 댓글 textarea 포커스

* feat: 더보기 버튼 왼쪽에 공유 버튼 추가 (클릭 시 토스트 메시지 등장)

* chore: color 및 크기 수정

* feat: hover 시 버튼 처리

* chore: textarea focus 시 처리

* feat: 모임 정보 안내 버튼 추가

* feat: 모임 정보 안내 버튼 안에 아이콘 추가

* feat: 모임 소개 말줄임 처리

* refactor: 코드 리뷰 반영

* feat: 댓글 입력 부분 디자인 수정사항 반영
@pull-request-size pull-request-size bot added size/L and removed size/XL labels Dec 9, 2023
@eunsukimme eunsukimme merged commit a6eb4a0 into develop Dec 9, 2023
1 check passed
@eunsukimme eunsukimme deleted the feat/#596 branch December 9, 2023 15:01
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.

피드 상세 뷰에 해당 모임 피드 리스트 추가
2 participants