Skip to content

Conversation

@jjeongdong
Copy link
Member

@jjeongdong jjeongdong commented Jan 31, 2025

#️⃣ 요약 설명

템플릿 타입에 맞는 커리어 조회 API 수정

📝 작업 내용

  • LoginMember 어노테이션을 사용하여 Member 객체를 받아오도록 수정했습니다.
  • Answer 조회 시 멤버의 Job 조건 추가
    • 사용자가 프로필에서 Job을 변경한 경우, 이전 Job에 대한 Answer가 조회되지 않도록 쿼리에 member.job 조건을 추가했습니다.

동작 확인

기능을 실행했을 때 정상 동작하는지 여부를 확인하고 사진을 올려주세요

ex) 테스트 코드 작성후 성공 사진
ex) swagger 사진

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@jjeongdong jjeongdong added the fix 버그/에러 수정 label Jan 31, 2025
@jjeongdong jjeongdong self-assigned this Jan 31, 2025
Copy link
Member

@momuzzi momuzzi left a comment

Choose a reason for hiding this comment

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

LGTM~~

Copy link
Contributor

@jpark0506 jpark0506 left a comment

Choose a reason for hiding this comment

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

LGTM~ 고생하셨습니다!

@jjeongdong jjeongdong merged commit e78dd60 into develop Jan 31, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 버그/에러 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants