Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🛰️ Issue Number
🪐 작업 내용
📚 Reference
✅ Check List
🔉 같이 고민하고 싶은 부분 (없으면 생략)
기존 최신순 콘텐츠 조회 api를 고도화 하였음(파라미터 추가해서 0이면 최신순, 1이면 좋아요(찜한) 콘텐츠 조회)
facade클래스의 option별 if-else문을 전략패턴으로 바꿔볼까 했는데 일단 옵션이 두 개 뿐이라 그대로 두었습니다.(나중에 기획에서 좀 더 살이 붙어서 더 추가될 여지가 있으면 해당패턴으로 리팩토링하겠습니다.)