Conversation
jeongkyueun
left a comment
jeongkyueun
left a comment
There was a problem hiding this comment.
기존 API에 JWT를 반영하시고 구글 소셜 로그인 구현 잘 하신 거 같습니다 수고 많으셨습니다!!!
soyun0318
left a comment
soyun0318
left a comment
There was a problem hiding this comment.
수정 사항에 맞춰 swagger까지 잘 수정하신 것 같아요
KateteDeveloper
left a comment
KateteDeveloper
left a comment
There was a problem hiding this comment.
서비스까지 api에 맞춰서 수정해주셨습니다! 너무 수고 많으셨습니다:)
ywkim1m
left a comment
ywkim1m
left a comment
There was a problem hiding this comment.
구글 로그인 구현 잘해주신 것 같습니다! 수고하셨습니다!!
suyeon0421
left a comment
suyeon0421
left a comment
There was a problem hiding this comment.
미션 깔끔하게 작성 너무 잘하신 것 같아요~~!!! 수고하셨습니다!!
Hminkyung
left a comment
Hminkyung
left a comment
There was a problem hiding this comment.
9주차 미션 깔끔하게 정리 잘해주신 것 같아요!! 수행하느라 너무너무 고생많으셨어요!! ㅎㅎ 이제 배포해보면서 한주만 더하면 끝이니까 마지막까지 화이팅 해봅시다!! 고생하셨어요~
| }; | ||
|
|
||
| // 내 리뷰 조회 | ||
| export const handleListMyReviews = async (req, res, next) => { |
There was a problem hiding this comment.
index.js에서 라우터 처리하고 내 리뷰 조회는 JWT를 사용하고 있어요! 따라서 여기서는 userId를 쿼리로 받을 필요 없어요! JWT를 적용했기 때문에 현재 로그인한 내 리뷰가 되어있는 상황이라서 userId 쿼리는 필요없어요!
- 추가적으로 ValidationError가 import 되어있지 않아요!! import 가 되어있지 않으면 not defined error가 뜹니다!
There was a problem hiding this comment.
쿼리에서 userId 를 받지 않고 JWT payload에서 가져오는 형식으로 진행하면 좋고 Swagger 주석도 마찬가지로 userId 쿼리 파라미터 설명은 제거해도 됩니다!
| } | ||
| */ | ||
| try { | ||
| const userId = Number(req.params.userId); |
There was a problem hiding this comment.
isLogin으로 JWT 인증은 걸려있지만 실제 조회 대상이 :userId path 파라미터 형식으로 되어있어요 ! 로그인한 유저가 path userId가 아니더라도 요청이 통과되고 있는데 즉, 내가 로그인만 되어있다면 다른 유저의 미션 목록도 조회가 가능하다 라는 의미가 되어버립니다! 따라서 내 미션 목록 조회니까 const userId = req.user.ud 식으로 진행이 되어야 합니다.
| name: profile.displayName, | ||
| gender: null, | ||
| birth: new Date(1970, 0, 1), | ||
| address: "추후 수정", |
There was a problem hiding this comment.
추후 수정이라는 값이 DB에 남는 것보다는 빈 문자열 혹은 각각에 어떤 식으로 들어갈 지 고려해서 넣는것도 좋을 것 같아요!
예를 들어서 gender의 경우는 단순 null 보다는 userSignUp과 upadateMyProfile 쪽과 gender가 동일해야 하니까 어떤 값으로 관리할지 고민해볼수 있겠죠!
7 participants
🚀 작업한 기능 설명 (Feature Description)
🔍 작업 상세 (Implementation Details)
🖼️ 이미지 첨부 (Images)
📋 관련 자료 (Related Resources)
📝 추가 정보 (Additional Information)