Skip to content

Conversation

@HiJuwon
Copy link
Collaborator

@HiJuwon HiJuwon commented Aug 21, 2025

체크리스트

  • 🧰 npm run dev로 실행 환경에서 잘 돌아가는걸 확인했나요?
  • 🎋 base 브랜치를 develop 브랜치로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙆 리뷰어는 등록했나요?

📌 관련 이슈번호


✅ Key Changes

기존 문제점

  • 새로 목표/이슈를 생성할 때, 혹은 이미 생성 완료된 목표/이슈를 조회할때
  • 어느 때에 상관없이 상세페이지 상단 헤더(DetailHeader, WorkspaceDetailHeader)의 id가 다음 생성될 목표/이슈의 id로 나타나는 문제가 있었습니다.

원인 파악

  • 목표/이슈/외부 이름 관련 쿼리키 무효화를 잘못 설정한 줄 알았으나, 이러한 부분의 문제는 아니었고
  • 상세페이지 헤더(DetailHeader, WorkspaceDetailHeader) 내부에서 항상 다음 생성될 목표/이슈/외부 이름을 조회하는 API 훅을 호출하고, 그로부터 받은 이름을 id값으로 적용하고 있었기 때문에 무조건 생성될 id가 뜨고 있었습니다.

오류 해결

  • 상세페이지 헤더(DetailHeader, WorkspaceDetailHeader) 내부에서 생성될 목표/이슈/외부 이름을 조회하는 API 훅을 호출하는 로직을 제거하고, props로 부모 컴포넌트로부터 받아올 detailId를 추가했습니다.
  • 상세페이지 헤더 컴포넌트들: 단순 헤더 값 렌더링만 담당하는 것으로 컴포넌트 역할을 축소했습니다.
  • 실제 상세페이지 id값: 각 상세페이지 파일들(GoalDetail, IssueDetail, ExternalDetail 등)에서 불러옵니다.
    • (1) 생성 모드일 때: 생성될 목표/이슈/외부 이름 조회하는 API 훅으로부터 id값을 받아와 detailId로 연결.
    • (2) 생성 모드가 아닐 때: 각 목표/이슈/외부 상세 조회 API 훅의 응답값으로 받는 name을 불러와 detailId로 연결.

📸 스크린샷 or 실행영상

(1) 해당 팀내의 목표 목록을 보면 기존 g1~g6까지의 목표들이 생성되어있었음

스크린샷 2025-08-22 오전 3 56 36

(2) 새로 목표를 생성하면 생성될 목표 이름 조회 API 훅을 통해, 신규 목표 id(g7)이 나타남

스크린샷 2025-08-22 오전 3 56 41

(3) 목표 생성 완료 후 다시 해당 목표로 돌아와도: 다음 생성될 목표 id(g8)이 뜨는 오류는 사라지고, 기존 id(g7) 그대로 보임

스크린샷 2025-08-22 오전 3 57 27

(4) 기존에 생성된 목표의 경우에도 기존 id값(g5)이 제대로 보임

스크린샷 2025-08-22 오전 3 57 31

💬 To Reviewers

@HiJuwon HiJuwon self-assigned this Aug 21, 2025
@HiJuwon HiJuwon added the 🚨 FIX 버그나 오류 수정 label Aug 21, 2025
@HiJuwon HiJuwon requested a review from gaeulzzang as a code owner August 21, 2025 19:08
@HiJuwon HiJuwon added the ♻️ REFACTOR 코드 수정 및 리팩토링 label Aug 21, 2025
@HiJuwon HiJuwon requested a review from waldls as a code owner August 21, 2025 19:08
@github-actions
Copy link

Copy link
Collaborator

@jinj00oo jinj00oo 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
Collaborator

@sunhwaaRj sunhwaaRj left a comment

Choose a reason for hiding this comment

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

쿼리키때문이 아니었네요.....짱

@sunhwaaRj sunhwaaRj merged commit 34c20bc into develop Aug 22, 2025
1 check passed
@HiJuwon HiJuwon deleted the feature/#198-detail_querykey-refactor branch August 22, 2025 01:15
@HiJuwon HiJuwon added this to the Demo Day milestone Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ REFACTOR 코드 수정 및 리팩토링 🚨 FIX 버그나 오류 수정 🐥 주원

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 상세페이지 쿼리키 무효화 점검 및 상세페이지 id 오류 해결

4 participants