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: 마이스페이스 내가 받은 픽 API #87

Merged
merged 7 commits into from
Aug 16, 2024

Conversation

toychip
Copy link
Member

@toychip toychip commented Aug 12, 2024

  • 🔀 PR 제목의 형식을 잘 작성했나요? e.g. [add] pr template
  • 🧹 불필요한 코드는 제거했나요?

작업 내용

  1. 기존 feat: Create QuestionSheet #86 에서 작업을 시작했습니다.
  2. 마이스페이스에서 내가 받은 픽을 조회하는 API 만들었습니다.
  3. 공동 등수를 고려하여 계산 후 반환합니다.

비고 (첨부자료)

이번 리뷰도 잘 부탁드립니다. ☺️

@toychip toychip self-assigned this Aug 12, 2024
@toychip toychip force-pushed the junhyoung/add-myspace-mypick branch from 18ff5a4 to 9b81c2d Compare August 14, 2024 06:58
Comment on lines +217 to +220
.orderBy(
Wildcard.count.desc(),
pickEntity.createdAt.desc()
)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wildcard.count 는 첨보는 형태네요 신기하네용

Comment on lines 139 to 168
override fun receivedMySpacePicks(): List<PickService.MySpacePickDetail> {
// ToDo 실제 로그인 값으로 사용
val currentMemberId = MemberId("currentMemberId")
val mySpacePicks = pickService.getReceivedMySpacePicks(currentMemberId)
Copy link
Collaborator

Choose a reason for hiding this comment

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

이제 실제 로그인 값(멤버 아이디) 불러올 수 있어서 Todo 로 안박아도 될것 같아요!

@@ -7,4 +7,5 @@ spring:
dojo:
candidate:
size: 8

rank:
Copy link
Member

Choose a reason for hiding this comment

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

요것도 api 밑에 yaml 로 옮겨주세요~

@toychip toychip force-pushed the junhyoung/add-myspace-mypick branch from c4151b3 to 282973c Compare August 16, 2024 06:35
@toychip toychip merged commit b37e7d5 into master Aug 16, 2024
1 check passed
@toychip toychip deleted the junhyoung/add-myspace-mypick branch August 16, 2024 06:54
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.

3 participants