Skip to content

Conversation

@yiseungyun
Copy link
Member

@yiseungyun yiseungyun commented Nov 28, 2024

관련 이슈 번호

  • 작성자: 이승윤
  • 작성 날짜: 2024.11.28

✅ 체크리스트

  • 코드가 정상적으로 작동하는지 확인했습니다.
  • 주요 변경사항에 대한 설명을 작성했습니다.
  • 코드 스타일 가이드에 따라 코드를 작성했습니다.

🧩 작업 내용

  • 세션 폼 생성 시 모달창 안뜨는 오류 수정
  • 마이페이지 질문지 탭 실제 api 연결
  • 질문지 카드 삭제 시 실시간 반영되도록 구현

📝 작업 상세 내역

세션 폼 생성 시 모달창 안뜨는 오류 수정

  • 기존에 모달 리팩토링으로 인해 이전 코드가 남아있던 문제
  • 다시 새로운 방법 적용 후 모달창 이용가능하게 수정

마이페이지 질문지 탭 실제 api 연결

  • 페이지네이션 메타 데이터 이용해 정확히 설정
  • 실제 데이터 가져오기

질문지 카드 삭제 시 실시간 반영

  • 반영이 안되는 문제가 있었는데 동일한 쿼리키 무효화시켜주도록 변경
    • 추후 삭제되는 탭 확인 후 해당 쿼리만 무효화할 수 있는지 확인해야할 거 같다.

💬 다음 작업 또는 논의 사항

  • 실제 api 연결해서 데이터 표시하면서 발견한건데 마이페이지에 질문지 개수나 혹은 질문 내용에 따라 너비가 달라지는 문제가 있어요. 이 부분 수정하겠습니다.
  • 삭제 쿼리까지는 했는데 수정은 아직 구현이 안된 상태입니다. 수정 페이지를 생성 후 해야해서 이후 작업을 해야합니다.

Copy link
Member

@ShipFriend0516 ShipFriend0516 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
Member

Choose a reason for hiding this comment

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

확실히 질문지 리스트 불러오는 API같이 자주 사용되는 것들은 이렇게 분리하는게 유리한 것 같심

Comment on lines +82 to 86
<ListSelectModal modal={modal} />
<CategorySection />
<TitleSection />
<QuestionListSection />
<QuestionListSection openModal={modal.openModal} />
<ParticipantSection />
Copy link
Member

Choose a reason for hiding this comment

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

useModal로 생성한 modal은 모달로 만들고 싶은 컴포넌트에 전달하고, 모달을 여는 버튼에는 modal.openModal을 전달하는 방식이군요 활용해보겠슴니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants