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

스웨거 문서 수정 / 인기 키워드 조회 버그 수정 #24

Merged
merged 7 commits into from
Jul 6, 2024

Conversation

seohyun0120
Copy link
Collaborator

개선

  • 모든 라우트에 loggerMiddleware 적용
  • Meme 라우터에서 UserMiddleware 우선 통과하도록 변경 (user -> meme validation)
  • swagger 개선
    • 자동 생성으로 잘못 들어간 필드값 변경
    • 200 이외 응답에서 data 필드 없는 예제 수정
    • 기타 등등

버그

  • 밈 생성, 수정 시 keywordIds 를 string[] 으로 넣어주고있는 문제 수정 ObjectId[] 여야함 -> dev db는 수정해둠
  • 인기 키워드 조회 API에서 키워드에 해당하는 밈 조회 쿼리 수정 (keywordIds과 완벽 일치하는지, string 비교로 발생한 문제)

@seohyun0120 seohyun0120 requested review from pjw5521 and Hyun-git July 6, 2024 15:42
@seohyun0120 seohyun0120 merged commit b6ba6b1 into develop Jul 6, 2024
1 check passed
@seohyun0120 seohyun0120 deleted the feature/update-swagger-docs branch July 6, 2024 18:14
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.

1 participant