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

[Feature] 이동봉사 후기 조회 API 수정 #170

Merged
merged 13 commits into from
May 4, 2024

Conversation

hojeong2747
Copy link
Member

@hojeong2747 hojeong2747 commented May 2, 2024

💡 연관된 이슈

close #168

📝 작업 내용

  • 후기 단건 조회 API 반환값 수정 (후기 작성일, 모집자 아이디, 공고 메인이미지, 공고 아이디 필드 추가)
  • 후기 전체 조회 API 반환값 수정 (후기 작성일, 모집자 아이디, 공고 메인이미지, 공고 아이디 필드 추가)
  • 후기 단건 조회 API 테스트 코드 수정
  • 후기 전체 조회 API 테스트 코드 수정

  • 모집자 프로필 - 후기 조회 API 반환값 수정 (후기 작성일, 모집자 아이디, 공고 메인이미지, 공고 아이디 필드 추가)
  • 모집자 프로필 - 후기 조회 API 테스트 코드 수정

후기 관련 API 총 6개 정리

  1. 봉사자 후기 단건 조회 [GET] /volunteers/reviews/{reviewId}
  2. 모집자 후기 단건 조회 [GET] /intermediaries/reviews/{reviewId}
  3. 봉사자 후기 전체 조회 [GET] /volunteers/reviews
  4. 봉사자 후기 등록 [POST] /volunteers/posts/{postId}/review
  5. 봉사자 모집자 프로필 - 후기 조회 [GET] /volunteers/intermediaries/{intermediaryId}/reviews
  6. 모집자 프로필 - 후기 조회 [GET] /intermediaries/{intermediaryId}/reviews

💬 리뷰 요구 사항

반환값 수정과 테스트 코드 수정만 하면 되는 간단한 작업이지만 혼자 고군분투하게 한 오류 공유합니다. 오랜만에 코드 적다 보니 빠르게 원인을 캐치하지 못하더라구요🥲

  1. post.mainImage.image로 String 값을 가져와야 하는데 post.mainImage로만 가져와서 DTO 생성자와 필드 타입이 맞지 않아 매핑이 안되는 문제
  2. LocalDateTime 형식의 createdDate 값을 LocalDate로 포맷팅 하던 중 -> record 클래스에서 뫼비우스의 띠처럼 LocalDate <-> LocalDate 변환만 계속했던 문제 (처음 오류 원인이 여기인 줄 알았는데 알고보니 1번이 원인이었다! gpt 3.5는 정말 babo인데 그 말을 듣고 내 코드를 의심한 내가 조금 더 babo스러웠다. 반성했어요)

추가로 이번 코넥독 2.0에서는 후기에 사진 여러 장을 업로드하고 조회할 수 있게 되어 추가 구현을 하려던 찰나, 기존 코드를 살펴보니 이미 여러 장 업로드, 조회할 수 있게 구현했더라구요. 약간 뿌듯했던 순간도 공유합니다😉

@hojeong2747 hojeong2747 added ✨ Feature 기능 개발 Priority: Medium 우선순위 중간 🐰 Hojeong 담당자 labels May 2, 2024
@hojeong2747 hojeong2747 self-assigned this May 2, 2024
Copy link
Member

@kyeong-hyeok kyeong-hyeok left a comment

Choose a reason for hiding this comment

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

좋습니다!👍 후기 조회 시 공고를 타고 들어갈 수 있다고 알고 있어서 그 부분만 확인해서 수정해 주시면 될 것 같아요!

@JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd", timezone = "Asia/Seoul")
LocalDate startDate,
@JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd", timezone = "Asia/Seoul")
LocalDate endDate,
String departureLoc, String arrivalLoc,
String intermediaryName, String content
Long intermediaryId, String intermediaryName
Copy link
Member

Choose a reason for hiding this comment

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

공고 부분을 클릭했을 때 해당 공고로 가야하기 때문에 postId 필드도 추가되어야 할 것으로 보입니다! 확인 부탁드려요

Copy link
Member Author

Choose a reason for hiding this comment

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

확인 감사합니다 :) 역시 꼼꼼

String departureLoc, String arrivalLoc,
String intermediaryName, String content
) {
Long intermediaryId, String intermediaryName
Copy link
Member

Choose a reason for hiding this comment

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

이 부분도 위에서 답변한 내용 참고해 주시면 감사하겠습니다! (postId)

@kyeong-hyeok
Copy link
Member

모집자 프로필에서 후기를 조회할 때 봉사자, 모집자 모두 동일한 화면을 보게 된다면 이렇게 uri로 구분하는 거 좋은 것 같습니다! 👍
만약 서로 보는 화면에 차이가 생긴다면 그때 나눠도 될 거 같아요 ㅎㅎ 수고하셨습니다~

@hojeong2747 hojeong2747 merged commit 44d4e73 into develop May 4, 2024
1 check passed
@hojeong2747 hojeong2747 deleted the feat/168-review-read-api branch May 4, 2024 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🐰 Hojeong 담당자 Priority: Medium 우선순위 중간
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 이동봉사 후기 조회 API 수정
2 participants