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.

Sign up for GitHub

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

Jump to bottom

[BE] 리뷰 단건 조회 시, 리뷰이 이름으로 치환되지 않는 오류 수정 #327

Merged
merged 2 commits into from
Aug 13, 2024
Merged

[BE] 리뷰 단건 조회 시, 리뷰이 이름으로 치환되지 않는 오류 수정 #327

merged 2 commits into from
Aug 13, 2024

Conversation

skylar1220
Copy link
Contributor

@skylar1220 skylar1220 commented Aug 12, 2024 •
edited
Loading

🚀 어떤 기능을 구현했나요 ?

  • 리뷰 단건 조회 시, 리뷰이 이름으로 치환되지 않는 오류를 수정했습니다.

🔥 어떻게 해결했나요 ?

  • 리뷰 단건 조회 응답 매핑 시 필요한 부분을 리뷰이 이름으로 변환하는 로직으로 대체했습니다.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 치환해야하는 데 안빼먹었는지 확인!

📚 참고 자료, 할 말

  • 올리가 알려줘서 반영하고 바로 PR 올립니당~

@skylar1220 skylar1220 added this to the 4차 스프린트: 최종 milestone Aug 12, 2024
@skylar1220 skylar1220 self-assigned this Aug 12, 2024
@skylar1220 skylar1220 linked an issue Aug 12, 2024 that may be closed by this pull request
[BE] 리뷰 단건 조회 시, 리뷰이 이름으로 치환되지 않는 오류를 수정한다. #326
Closed
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 12, 2024 •
edited
Loading

Test Results

58 tests   58 ✅  2s ⏱️
22 suites   0 💤
22 files     0 ❌

Results for commit 2bec73b.

♻️ This comment has been updated with latest results.

@ImxYJL
Copy link
Contributor

ImxYJL commented Aug 13, 2024

고쳐줬구나. 든든하다.

@donghoony
Copy link
Contributor

공통 관심사라 객체로 분리해야겠습니다.. 방법을 탐구해 보아요

nayonsoso
nayonsoso approved these changes Aug 13, 2024
View reviewed changes
Copy link
Contributor

@nayonsoso nayonsoso left a comment

Choose a reason for hiding this comment

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

좋습니당~

@skylar1220 skylar1220 merged commit a075682 into develop Aug 13, 2024
5 checks passed
@donghoony donghoony deleted the be/fix/326-fix-reviewee-name-parsing branch August 20, 2024 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@donghoony donghoony donghoony approved these changes

@nayonsoso nayonsoso nayonsoso approved these changes

@Kimprodp Kimprodp Kimprodp approved these changes

Assignees

@skylar1220 skylar1220

Labels
✨ BE 🐛 버그
Projects
2024-review-me
Status: Done
Milestone
4차 스프린트: 최종
Development

Successfully merging this pull request may close these issues.

[BE] 리뷰 단건 조회 시, 리뷰이 이름으로 치환되지 않는 오류를 수정한다.
5 participants
@skylar1220 @ImxYJL @donghoony @nayonsoso @Kimprodp