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

[FE] refactor: 리뷰 상세 페이지 url path 관련 코드 수정 및 리팩터링 #321

Merged
merged 6 commits into from
Aug 12, 2024

Conversation

soosoo22
Copy link
Contributor

@soosoo22 soosoo22 commented Aug 12, 2024


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

  • 리뷰 상세 페이지 url path 관련 코드를 수정하고 리팩터링했습니다.

🔥 어떻게 해결했나요 ?

리뷰 상세 페이지에서 리뷰를 불러오는 API가 변경되어 memberId 가 더이상 필요하지 않게 되었습니다. 그래서 memberId 관련 코드를 모두 제거했습니다.

변경사항

  • getDetailedReviewApi와 관련된 코드에서 memberId 파라미터를 제거하고, 관련된 모든 사용처에서도 memberId를 삭제했습니다.
  • 목 데이터와 핸들러에서 memberId 관련 로직을 제거했습니다.
  • ReviewListPage에서 리뷰 클릭 시, navigate 경로에 memberId를 제거했습니다.

리팩터링

  • REVIEW_MESSAGE_SUFFIX에서 님을 제외하여 "에게 달린 리뷰입니다!"로 수정했습니다.
  • 리뷰 상세 페이지 목 데이터에서 REVIEWEENAME 상수를 대문자로 변경했습니다.
    리뷰 상세 페이지 개선
    • key 값을 questionId로 변경했습니다.

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

  • memberId 관련 코드가 모두 삭제되었을까요?

📚 참고 자료, 할 말

  • 리뷰미 리뷰미 화이팅!

soosoo22 and others added 3 commits August 12, 2024 17:27
Co-Authored-By: badahertz52 <[email protected]>
Co-Authored-By: ImxYJL <[email protected]>
Co-Authored-By: Fe <[email protected]>
Co-Authored-By: badahertz52 <[email protected]>
Co-Authored-By: ImxYJL <[email protected]>
Co-Authored-By: Fe <[email protected]>
@soosoo22 soosoo22 linked an issue Aug 12, 2024 that may be closed by this pull request
1 task
soosoo22 and others added 2 commits August 12, 2024 18:17
Co-Authored-By: badahertz52 <[email protected]>
Co-Authored-By: ImxYJL <[email protected]>
Co-Authored-By: Fe <[email protected]>
Co-Authored-By: badahertz52 <[email protected]>
Co-Authored-By: ImxYJL <[email protected]>
Co-Authored-By: Fe <[email protected]>
@soosoo22 soosoo22 changed the title [FE] refactor: 리뷰 상세 페이지 url path 관련 코드를 수정 [FE] refactor: 리뷰 상세 페이지 url path 관련 코드 수정 및 리팩터링 Aug 12, 2024
Co-Authored-By: badahertz52 <[email protected]>
Co-Authored-By: ImxYJL <[email protected]>
Co-Authored-By: Fe <[email protected]>
@soosoo22 soosoo22 requested review from BadaHertz52, chysis and ImxYJL and removed request for chysis August 12, 2024 09:26
Copy link
Contributor

@chysis chysis left a comment

Choose a reason for hiding this comment

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

코멘트 반영된 것 확인했습니다 고생했어요!👍

@soosoo22 soosoo22 self-assigned this Aug 12, 2024
Copy link
Contributor

@ImxYJL ImxYJL left a comment

Choose a reason for hiding this comment

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

URL 뜯어고치느라 수고하셨어요~~

Copy link
Contributor

@ImxYJL ImxYJL left a comment

Choose a reason for hiding this comment

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

다시 어프루브함니다

Copy link
Contributor

@BadaHertz52 BadaHertz52 left a comment

Choose a reason for hiding this comment

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

고생했어!!!!

@soosoo22 soosoo22 merged commit f769ac1 into develop Aug 12, 2024
4 checks passed
@soosoo22 soosoo22 added this to the 4차 스프린트: 최종 milestone Aug 12, 2024
@donghoony donghoony deleted the fe/refactor/319-detailed-review-page-updated-path branch August 20, 2024 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

[FE] 리뷰 상세 페이지 url path 관련 코드를 수정한다.
4 participants