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

✨ 피드 북마크 API 연동 #52

Merged
merged 15 commits into from
May 13, 2024
Merged

Conversation

BangDori
Copy link
Collaborator

@BangDori BangDori commented May 13, 2024

작업 이유

  • 피드 북마크 API Mocking 추가
  • 피드 북마크 API 연동
  • FeedsQueryData 공용 타입 설정

작업 사항

1️⃣ 피드 북마크 API Mocking 추가

기존에 피드 북마크 API가 추가되어 있지 않아서 피드 북마크 API handler 추가하였습니다. (wiki 반영 - Bookmark API Specification)

  • 피드 북마크 API
  • 피드 북마크 취소 API
  • 기존에 북마크 여부를 나타내는 변수명을 수정하였습니다. (isBookmark -> isBookmarked)

2️⃣ 피드 북마크 API 연동

  • 북마크 API의 경우 기존에 좋아요 API와 동일하게 MutationFn을 조건부로 처리하였습니다.
mutationFn: () =>
  isBookmarked
    ? requestUnbookmarkFeed(feedId)
    : requestBookmarkFeed(feedId),
  • 좋아요 API와 마찬가지로 사용자에게 빠른 사용자 경험을 제공해주기 위해 낙관적 업데이트 기능을 추가해두었습니다.
  • 북마크 상태에 따라 북마크 API 혹은 북마크 취소 API 호출이 정상적으로 이루어지는지에 대한 테스트 코드를 추가하였습니다.

image

3️⃣ FeedsQueryData 공용 타입 설정

export interface FeedsQueryData {
  queryParams: number[];
  pages: FetchFeeds[];
}

좋아요 API에서 낙관적 업데이트를 위해 사용되던 타입이 북마크 API에도 사용되어, 이를 공용 폴더에 위치시켰습니다.


리뷰어가 중점적으로 확인해야 하는 부분

  • 북마크 및 북마크 취소 API가 정상적으로 동작하고 있나요?
  • useBookmarks에서 개선할 부분이 있나요?
  • useBookmarks에 대해 추가로 고려해야 할 테스트가 있나요?

발견한 이슈

  • 없음

@BangDori BangDori self-assigned this May 13, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-52.d37mn03xh3qyyz.amplifyapp.com

Copy link
Member

@Legitgoons Legitgoons left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다. 병합하셔도 좋습니다!

src/features/feed-bookmark/test/useBookmarks.test.tsx Outdated Show resolved Hide resolved
@BangDori BangDori merged commit 2ca4aae into main May 13, 2024
2 checks passed
@BangDori BangDori deleted the feature/PW-298-feed-bookmark branch May 13, 2024 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants