✨ 피드 신고 UI 반영

[BangDori]

작업 이유

• ✨ 피드 신고하기 UI 수정 #65 이슈 처리

작업 사항

1️⃣ 숨김 관리 저장소 타입 수정

기존에 숨김 관리 저장소에서 관리하고 있던, hiddenFeeds의 타입을 다음과 같이 변경하였습니다.

interface HiddenFeedState {
  hiddenFeeds: Map<number, HiddenType>;
}

• 각 유형(숨김 처리 / 신고 처리)에 해당하는 메시지를 보여주기 위해 HiddenType을 적용하였습니다.
  • HiddenType이 hidden일 경우 숨김 처리가 적용 된 피드에 대한 메시지와 취소 버튼이 렌더링됩니다.
  • HiddenType이 siren일 경우, 신고 처리가 적용 된 피드에 대한 메시지와 취소 버튼이 렌더링됩니다.

2️⃣ mutation 방식 수정

기존에 비동기 함수로 생성하여 await로 작업(숨김 처리 / 신고 처리)이 완료될 때 까지 대기한 후 숨김 처리가 되는 방식이였는데, 숨김 및 신고 처리의 경우에 응답을 대기할 필요 없이 사용자에게 UI를 반영하는 것이 좋을 것 같다는 생각을 하여 await를 제거하였습니다.

• mutateAsync -> mutate

3️⃣ 피드 신고 API

• 피드 신고 API가 호출된 이후 해당 신고가 접수되었다는 메시지를 표시하기 위해, handleMutationSuccess 함수를 생성하였습니다.
• 피드 신고 API가 완료된 이후 isBlind라면 숨김 처리가 되도록 수정하였습니다.

4️⃣ 피드 신고하기 UI 수정

• ✨ 피드 신고하기 UI 수정 #65

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

• 숨김 관리 저장소 타입이 적절하게 수정되었나요?

발견한 이슈

• 테스트 코드 작성과 관련한 이슈가 발생
  • 유닛 테스트와 통합 테스트의 경계가 애매모호함
  • 유닛 테스트가 유닛 테스트만의 역할을 수행하고 있지 않음

질문

현재 피드 신고 API가 완료된 이후 onSuccess에서 isBlind를 확인하고 숨김 처리가 되도록 하고 있어, 응답이 돌아올 때 까지 기다리고 있습니다. 이는 await를 제거한 것과 관련없이 동일하게 서버의 응답이 돌아올 때 까지 대기하고 있는 것과 마찬가지라고 생각합니다.

그래서 onSuccess에서 처리하는 것이 아닌, 사용자의 의도에 맞게 Mutation이 호출되었을 때 바로 숨김 처리를 하는 방법에 대해서는 어떻게 생각하시나요?

[aws-amplify-ap-northeast-2]

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

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

[Legitgoons]

수고 많으셨습니다. 간단한 질문이 있어 작성해보았습니다.

[Legitgoons]

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