Skip to content

[Feat] 스터디 후기 모달 렌더링 로직 추가#127

Merged
aken-you merged 12 commits intodevelopfrom
feat/study-review
Aug 29, 2025
Merged

[Feat] 스터디 후기 모달 렌더링 로직 추가#127
aken-you merged 12 commits intodevelopfrom
feat/study-review

Conversation

@aken-you
Copy link
Contributor

@aken-you aken-you commented Aug 27, 2025

☘️ 작업 내용

스터디 후기 모달 렌더링

스터디 후기 모달이 렌더링되는 기준은 아래와 같습니다.

  1. 스터디 후기를 안달았을 경우
  2. 토~일요일에 30분 주기로 보여줌

useReviewReminder 훅은 스터디 후기 모달을 렌더링해야하는지 여부를 반환합니다.
주말이라면, 30분마다 서버로부터 후기 작성 여부 데이터를 요청합니다.
localStorage에 lastReviewModalShown이 없으면 모달을 렌더링하고, localStorage에 모달을 보여준 시각을 저장합니다.
만약 localStorage에 값이 있다면, 현재로부터 30분이 지났는지 확인합니다. 지났다면, 모달을 렌더링하고 localStorage에 저장된 값을 업데이트 해줍니다.

테스트 방법

  1. isWeekend 변수의 dayOfWeek을 오늘 요일 숫자값으로 변경 (useShouldReviewPartnerQuery, useReviewReminder)
  2. useShouldReviewPartnerQuery의 refetchInterval 짧게 설정
  3. useReviewReminder의 THIRTY_MIN을 짧게 설정

키워드 리스트 모달

아래 모달 구현했습니다

스크린샷 2025-08-28 오전 12 36 00

후기 작성 완료 모달 x

석환님에게 이 부분은 추후에 반영될 수 있다고 미리 말씀해놓은 상태입니다..
석환님께서 그정도는 괜찮다고 하셨는데, 이 부분은 우선순위가 낮은 것 같아 나중에 반영하도록 하겠습니다.
스크린샷 2025-08-28 오전 12 34 29

@aken-you aken-you self-assigned this Aug 27, 2025
Copy link
Contributor

@Mimiminz Mimiminz left a comment

Choose a reason for hiding this comment

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

리뷰 남길게 크게 없네요...ㅎvㅎ..
제 계정엔 리뷰 관련 데이터가 없어서 아직 확인이 어렵기도 하고..
수아님 계정으로 데이터 확인 하시고, 큰 문제 없다면 일단 성진님과 이야기 후 운영 서버에 반영해도 괜찮지 않을까 싶습니다..!

Comment on lines +84 to +88
addStudyReview(form, {
onSuccess: () => {
onClose();
},
});
Copy link
Contributor

Choose a reason for hiding this comment

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

이 파일이 스터디 종료 후 후기 작성 모달이 맞을까요...?
alert로 표기만 해두고 커스텀을 나중에 한다는 걸로 이해했었는데, 아직 알림 자체가 추가되지 않은 건가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

여기 훅에다가 두었습니다!

export const useAddStudyReviewMutation = () => {
  return useMutation({
    mutationFn: addStudyReview,
    onSuccess: () => {
      // todo: 모달로 변경
      alert('후기 작성이 완료되었습니다.');
    },
  });
};

@aken-you aken-you merged commit 8bcf716 into develop Aug 29, 2025
2 checks passed
@aken-you aken-you mentioned this pull request Aug 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants