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

⚙️ MSW Feed, Comment, Like API 등록 #12

Merged
merged 20 commits into from
Apr 19, 2024
Merged

Conversation

BangDori
Copy link
Collaborator

@BangDori BangDori commented Apr 19, 2024

작업 이유

Feed, Comment, Like API에 대한 Mock API 등록


작업 사항

1️⃣ Feed, Comment, Like API 등록

  • Wiki API Specification 문서에 업로드 되어있는 API를 기반으로 추가하였습니다.

  • Like API에 대한 테스트 코드만 추가하였습니다.

    • 좋아요 버튼을 클릭하였을 때의 테스트
      image

    • 좋아요 API 예외 테스트
      image


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

MSW를 구현한 이유가 FE의 생산성을 향상을 위한 것이기에, 각각의 클래스로 연관성을 고려하여 구현하지 않았습니다. 이 점 유의하여 리뷰해주시면 감사하겠습니다. 😀

  • 누락된 예외 케이스가 있나요?
  • Wiki에 등록된 API 문서와 다른 점이 있나요?

발견한 이슈

1️⃣ MSW 요청 지연 처리

API 요청마다 첫 요청은 실패하는 문제가 발생하였습니다. 해당 문제를 확인해보니, 브라우저에 서비스 워커를 등록하는 과정이 비동기 작업이였는데 비동기 처리로 등록해주지 않아서 발생한 문제였습니다.

이에 비동기로 서비스 워커를 시작하도록 수정하였습니다.

2️⃣ Strict Mode 제거

API 요청시 StrictMode로 인해 불 필요한 요청이 발생하여 제거하였습니다.

@BangDori BangDori self-assigned this Apr 19, 2024
Copy link

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

Access this pull request here: https://pr-12.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.

Strict Mode 제거 인지하였습니다. 병합하셔도 좋습니다. 수고 많으셨습니다!
image

@BangDori BangDori merged commit 903875b into main Apr 19, 2024
3 checks passed
@BangDori BangDori deleted the feature/PW-260-feed-msw branch April 19, 2024 14:48
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