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: Add individual star rating searching func #120

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

selgyun
Copy link
Contributor

@selgyun selgyun commented Feb 15, 2022

개인별 별점 조회 기능을 만들었습니다.

포스트맨에서는
users/{user_id}/starRating
으로 조회할 수 있습니다.

@selgyun selgyun requested a review from celi1004 February 15, 2022 14:24
Comment on lines 12 to 13
import orangetaxiteam.cocoman.domain.PasswordEncoder;
import orangetaxiteam.cocoman.domain.PasswordValidator;
import orangetaxiteam.cocoman.domain.SocialInfoService;
import orangetaxiteam.cocoman.domain.SocialInfoServiceSupplier;
import orangetaxiteam.cocoman.domain.SocialProvider;
import orangetaxiteam.cocoman.domain.User;
import orangetaxiteam.cocoman.domain.UserRepository;
import orangetaxiteam.cocoman.domain.*;
Copy link
Contributor

Choose a reason for hiding this comment

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

여기 또 이렇게 됐다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗. 인텔리제이 설정 바꿨다.. 이제 와일드카드 없을듯!

Comment on lines 21 to 22
@Query(value = "SELECT s FROM StarRating s WHERE s.user.id = :user")
List<StarRating> findByUserId(Pageable pageable, String user);
Copy link
Contributor

Choose a reason for hiding this comment

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

query 안쓰고 만들 수 있을 것 같은데!
이런 식으로?!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정완료~

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