Feature/mission 09/현

[hyeeon]

🚀 작업한 기능 설명 (Feature Description)

• 8주차 피드백 반영
• 하드 코딩 삭제
• JWT 구현 및 적용
• 회원 정보 수정 API

🔍 작업 상세 (Implementation Details)

• 구글 소셜 로그인 구현
• 회원 정보 수정 API 생성
• 기존 하드코딩된 userId 로직 제거
• 인증이 필요한 API에 JWT 인증 적용

🖼️ 이미지 첨부 (Images)

• 

📋 관련 자료 (Related Resources)

• 기능 구현에 참고한 자료(링크, 문서 등)가 있다면 여기에 첨부해주세요.

📝 추가 정보 (Additional Information)

• 기능과 관련하여 추가적으로 고려해야 할 사항이나 특별한 요구사항이 있다면 여기에 작성해주세요.

[jeongkyueun]

기존 API에 JWT를 반영하시고 구글 소셜 로그인 구현 잘 하신 거 같습니다 수고 많으셨습니다!!!

[soyun0318]

수정 사항에 맞춰 swagger까지 잘 수정하신 것 같아요

[KateteDeveloper]

서비스까지 api에 맞춰서 수정해주셨습니다! 너무 수고 많으셨습니다:)

[ywkim1m]

구글 로그인 구현 잘해주신 것 같습니다! 수고하셨습니다!!

[suyeon0421]

미션 깔끔하게 작성 너무 잘하신 것 같아요~~!!! 수고하셨습니다!!

[Hminkyung]

9주차 미션 깔끔하게 정리 잘해주신 것 같아요!! 수행하느라 너무너무 고생많으셨어요!! ㅎㅎ 이제 배포해보면서 한주만 더하면 끝이니까 마지막까지 화이팅 해봅시다!! 고생하셨어요~

[Hminkyung]

index.js에서 라우터 처리하고 내 리뷰 조회는 JWT를 사용하고 있어요! 따라서 여기서는 userId를 쿼리로 받을 필요 없어요! JWT를 적용했기 때문에 현재 로그인한 내 리뷰가 되어있는 상황이라서 userId 쿼리는 필요없어요!

• 추가적으로 ValidationError가 import 되어있지 않아요!! import 가 되어있지 않으면 not defined error가 뜹니다!

[Hminkyung]

쿼리에서 userId 를 받지 않고 JWT payload에서 가져오는 형식으로 진행하면 좋고 Swagger 주석도 마찬가지로 userId 쿼리 파라미터 설명은 제거해도 됩니다!

[Hminkyung]

isLogin으로 JWT 인증은 걸려있지만 실제 조회 대상이 :userId path 파라미터 형식으로 되어있어요 ! 로그인한 유저가 path userId가 아니더라도 요청이 통과되고 있는데 즉, 내가 로그인만 되어있다면 다른 유저의 미션 목록도 조회가 가능하다 라는 의미가 되어버립니다! 따라서 내 미션 목록 조회니까 const userId = req.user.ud 식으로 진행이 되어야 합니다.

[Hminkyung]

추후 수정이라는 값이 DB에 남는 것보다는 빈 문자열 혹은 각각에 어떤 식으로 들어갈 지 고려해서 넣는것도 좋을 것 같아요!
예를 들어서 gender의 경우는 단순 null 보다는 userSignUp과 upadateMyProfile 쪽과 gender가 동일해야 하니까 어떤 값으로 관리할지 고민해볼수 있겠죠!