-
Notifications
You must be signed in to change notification settings - Fork 1
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/review now #40
Feat/review now #40
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.
잘 보고 갑니다 ..
}); | ||
|
||
// 변경 리뷰 저장 | ||
review.save(); |
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.
저도 담에 일케 할래요
src/controller/review.ts
Outdated
@@ -55,7 +57,59 @@ const postReviewBeforeController = async (req: Request, res: Response) => { | |||
res, | |||
returnCode.OK, | |||
true, | |||
"독서 중 등록 성공", | |||
"독서단계이동 (전 -> 중)", |
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.
이게 임시저장으로 쓰이면 전 -> 중은 아닌게 아닌가요? 🧐
src/service/review.ts
Outdated
}, | ||
}); | ||
|
||
// 2. 존재하지 않은 review |
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.
않은 ? 🧐🧐🧐 ? 🧐?
✅ PR check list
🌈 PR 요약
review now api 추가
📌 변경 사항
Linked Issue
close #28