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: readSharedExamInfo 구현 #257

Merged
merged 3 commits into from
Oct 20, 2024
Merged

Feat: readSharedExamInfo 구현 #257

merged 3 commits into from
Oct 20, 2024

Conversation

RinRinPARK
Copy link
Contributor

PR

✨ 작업 내용

  • readSharedExamInfo 구현

✨ 참고 사항


⏰ 현재 버그


✏ Git Close #256

@RinRinPARK RinRinPARK added ✨ Feature 기능 개발 ☘️ surin surin labels Oct 19, 2024
@RinRinPARK RinRinPARK self-assigned this Oct 19, 2024
Copy link

테스트 커버리지 측정

There is no coverage information present for the Files changed

Total Project Coverage 80.26% 🍏

@RinRinPARK RinRinPARK requested a review from adorableco October 19, 2024 08:15
@RinRinPARK RinRinPARK marked this pull request as draft October 19, 2024 08:17
@RinRinPARK RinRinPARK marked this pull request as ready for review October 19, 2024 08:18
Copy link

테스트 커버리지 측정

There is no coverage information present for the Files changed

Total Project Coverage 80.26% 🍏

endTime = sharedExam.endTime,
capacity = sharedExam.capacity,
workbookId = sharedExam.workbook.id,
isWriter = sharedExam.member.id == memberId,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
isWriter = sharedExam.member.id == memberId,
role =
if (exam.member.id == memberId) {
ParticipationType.CREATOR
} else {
ParticipationType.EXAMINEE
},

이거 readAllSharedExams 기능이랑 통일감있게 이렇게 수정하는 거 어떨까욘 ExamDto 에도 suggestion 넣어둘게

val endTime: LocalDateTime,
val capacity: Int,
val workbookId: UUID,
val isWriter: Boolean,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
val isWriter: Boolean,
val role: ParticipationType,

Comment on lines 365 to 373
return ReadSharedExamInfoResponse(
examId = examId,
title = sharedExam.title,
startTime = sharedExam.startTime,
endTime = sharedExam.endTime,
capacity = sharedExam.capacity,
workbookId = sharedExam.workbook.id,
isWriter = sharedExam.member.id == memberId,
)
Copy link
Contributor

Choose a reason for hiding this comment

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

여기서 공유용 시험을 생성한 사람이 조회하는거면 응시자 id , name 도 넘겨줘야할 것 같아요!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

클라에 정확히 머 필요한지 함 물어볼게!!

Copy link
Contributor

Choose a reason for hiding this comment

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

오오오키오키
readExamResultDetail 이 기능 쓰려면 저 정보가 어딘가엔 응답으로 들어가야할 것 같아스 ~ ~

Copy link
Contributor

Choose a reason for hiding this comment

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

이거 api가 따로 있었네연. ㅎ

Copy link

테스트 커버리지 측정

There is no coverage information present for the Files changed

Total Project Coverage 80.26% 🍏

@RinRinPARK RinRinPARK requested a review from adorableco October 19, 2024 15:12
Copy link
Contributor

@adorableco adorableco left a comment

Choose a reason for hiding this comment

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

좋아용👍🏻👼🏻🙌🤭

@RinRinPARK RinRinPARK merged commit 1958a79 into develop Oct 20, 2024
1 check passed
@RinRinPARK RinRinPARK deleted the rin/swm-198 branch October 20, 2024 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 ☘️ surin surin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ Feat: readSharedExamInfo 구현
2 participants