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

Feat/community : 커뮤니티/댓글/좋아요 진짜 90% 완성 #10

Merged
merged 6 commits into from
Nov 16, 2023

Conversation

seungueonn
Copy link
Contributor

PR 타입

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 업데이트
  • 기타 사소한 수정

개요

커뮤니티 (게시글/댓글/좋아요)

변경 사항

  • pathVariable 을 기존 한글에서 id 값으로 변경하였습니다! 이에 따라 content, keyword 서비스와의 어마무시한 통신이 예상 됩니다. 추후 시간이 된다면 비동기 통신이나 레디스 캐시 등을 꼭 .. 하고 싶습니다 !!!!
  • 기존 controller 단만 id로 변경하고 이후 repository단 로직은 string 유지하였습니다. 따라서 UseCase 단에서 keywordId -> keyword String 으로 변경해야하는데, 이게 UseCase단에서 이뤄지는게 .. 맞겠죠 ..?
  • 좋아요 로직도 변경했어요 하하! 좋은데?
  • contents service 랑 통신만 하면 돼요 !!!!!!!!!

코드 리뷰 시 참고 사항

테스트 결과

@seungueonn seungueonn merged commit b893d2b into master Nov 16, 2023
1 check passed
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