-
Notifications
You must be signed in to change notification settings - Fork 0
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] 홈 API 구현 #34
[FEAT] 홈 API 구현 #34
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
사소한 오타를 제외하고는 완벽한것 같아요-!
public class | ||
Note extends BaseTimeEntity { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[P2] 여기 불필요한 엔터가 들어간 것 같아요! 수정 부탁드림당~
@Query(value = "select b from Book b where b.isPopular = :isPopular") | ||
List<Book> getAllByPopular(@Param("isPopular") Boolean isPopular, PageRequest pageRequest); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
와우 Paging 쓰는거 처음보는데, 신기하군뇨~~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[P3] 혹시 여기 Book 뒤에 띄어쓰기 한번 더 들어간거 같은데 수정해주실 수 있나요~?
public Book(String uuid, String favoriteName, String favoriteImage, String title, String description, int backgroundColor, Member member, boolean isPopular) { | ||
this.uuid = uuid; | ||
this.favoriteName = favoriteName; | ||
this.favoriteImage = favoriteImage; | ||
this.title = title; | ||
this.description = description; | ||
this.backgroundColor = backgroundColor; | ||
this.member = member; | ||
this.isPopular = isPopular; | ||
} | ||
|
||
public static Book of(String uuid, String favoriteName, String favoriteImage, String title, String description, int backgroundColor, Member member) { | ||
return new Book(uuid, favoriteName, favoriteImage, title, description, backgroundColor, member); | ||
public static Book of(String uuid, String favoriteName, String favoriteImage, String title, String description, int backgroundColor, Member member, boolean isPopular) { | ||
return new Book(uuid, favoriteName, favoriteImage, title, description, backgroundColor, member, isPopular); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
필드 추가 확인했습니당~
|
||
@Component | ||
@RequiredArgsConstructor | ||
public class InitDb { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
InitDb 만드는거 귀찮은 작업인데,,, 맛있게 쓰겠습니다,,,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니당 ~~~~!!!👏👏
private final BookRepository bookRepository; | ||
|
||
public List<Book> getPopularBook() { | ||
return bookRepository.getAllByPopular(true, PageRequest.of(0, 6)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오호 PageRequest에 대해 새롭게 알고갑니다 !!!! 😁
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
헉 어프롭을 안했었네요 고고~~
# Conflicts: # src/main/java/org/sopt/lequuServer/domain/book/service/BookService.java # src/main/java/org/sopt/lequuServer/global/exception/enums/SuccessType.java
📌 관련 이슈
closed #8
✨ 어떤 이유로 변경된 내용인지
🙏 검토 혹은 리뷰어에게 남기고 싶은 말