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

밈 저장 취소 시 성공했음에도 false로 반환되는 문제, 밈 조회 시 isSaved 필드가 제대로 동작하지않는 문제 수정 #34

Merged
merged 11 commits into from
Jul 23, 2024

Conversation

seohyun0120
Copy link
Collaborator

버그

  • 밈 저장 취소 API를 호출했을때, 저장 취소를 실패했음에도 200 상태코드와 data: false 가 넘어오는 문제가 있어서 수정
  • 저장되지않은 밈인데도 밈 조회 API에서 isSaved: true로 넘어오는 문제가 있어서 수정

개선

  • MemeInteraction 조회 시 상황에 따라 isDeleted 포함 여부를 변경할 수 있도록 인자값 추가
  • MemeInteraction 미들웨어를 추가해서 save interaction이 있는 밈인 경우만 서비스 레벨로 넘어가도록 개선

@seohyun0120 seohyun0120 requested review from pjw5521 and Hyun-git July 23, 2024 15:17
@seohyun0120 seohyun0120 changed the title 밈 저장 취소 시 성공했음에도 false로 반환되는 문제 수정, 밈 조회 시 isSaved 필드가 제대로 동작하지않는 문제 수정 밈 저장 취소 시 성공했음에도 false로 반환되는 문제, 밈 조회 시 isSaved 필드가 제대로 동작하지않는 문제 수정 Jul 23, 2024
@seohyun0120 seohyun0120 merged commit 5a8da60 into develop Jul 23, 2024
1 check passed
Copy link
Member

@Hyun-git Hyun-git left a comment

Choose a reason for hiding this comment

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

고생했옹ㅇ

@@ -44,6 +47,33 @@ export const getRequestedMemeInfo = async (
next();
};

// getRequestedMemeInfo, getRequestedUserInfo 미들웨어가 먼저 불려야함
export const getRequestedMemeSaveInfo = async (
Copy link
Member

Choose a reason for hiding this comment

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

NotDeleted가 들어가면 좋긴 할것 같아용! deleted된걸 조회하는 것도 있으니

@seohyun0120 seohyun0120 deleted the feature/fix-isSaved-issue branch August 4, 2024 10:26
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