Skip to content

Conversation

@Chiman2937
Copy link
Member

@Chiman2937 Chiman2937 commented Jan 16, 2026

📝 변경 사항

💡 SSE 로직 구조화

기존

  • SSE 연결, 연결 해제, 오류 처리, 데이터 처리 함수가 하나의 useEffect 내부에서 전부 처리

수정

  • SSE 연결, 연결 해제, 오류 처리, 데이터 처리 함수를 별도로 분리
  • SSE 연결 오류 발생 시 재 연결 시도하는 함수 추가
  • useConnectSSE 훅은 SSE 관련 기능만 제공하고, NotificationProvider에서 useEffect를 통해 연결/연결 해제 시도하도록 로직 수정
  useEffect(() => {
    connect();
    return () => {
      disconnect();
    };
  }, []);
  • 알림 type별 쿼리 무효화 구문 간소화 - 같은 무효화 로직을 공유하는 case 별로 묶어 구문 간소화 적용
          case 'GROUP_JOIN':
          case 'GROUP_LEAVE':
          case 'GROUP_JOIN_APPROVED':
          case 'GROUP_JOIN_REJECTED':
          case 'GROUP_JOIN_KICKED':
            if (data.group) {
              queryClient.invalidateQueries({ queryKey: groupKeys.detail(String(data.group.id)) });
            }
            break;

🔗 관련 이슈

Closes #


🧪 테스트 방법

  • 수동 테스트 검증(로컬 환경)
  • 유닛 테스트 검증
  • 통합 테스트 검증

📸 스크린샷 (선택)


📋 체크리스트

  • 관련 문서를 업데이트했습니다 (필요한 경우)
  • 테스트를 추가/수정했습니다 (필요한 경우)
  • Breaking change가 있다면 명시했습니다

💬 추가 코멘트


CodeRabbit Review는 자동으로 실행되지 않습니다.

Review를 실행하려면 comment에 아래와 같이 작성해주세요

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 16, 2026

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot requested a review from wooktori January 16, 2026 05:44
@github-actions
Copy link

github-actions bot commented Jan 16, 2026

🎭 Playwright Report

E2E Test가 성공적으로 완료되었습니다.

Test 요약 내용을 확인해주세요.

Status Build Log Updated (UTC)
✅ Ready View Build 2026-01-16 05:46:00

📊 Test Summary

  • ✅ Passed: 3
  • ❌ Failed: 0
  • ⏱️ Duration: 36.4s

📜 Test Details

✅ Passed Tests (3)
  • profile.test.ts (3)
    • [chromium] 존재하지 않는 프로필 페이지로 접속 시 404 redirect 되는 지 테스트
    • [firefox] 존재하지 않는 프로필 페이지로 접속 시 404 redirect 되는 지 테스트
    • [webkit] 존재하지 않는 프로필 페이지로 접속 시 404 redirect 되는 지 테스트

@github-actions
Copy link

github-actions bot commented Jan 16, 2026

🎨 Storybook Report

변경 사항이 없습니다

모든 Story가 이전 빌드와 동일합니다.

Status Storybook Build Log Updated (UTC)
✅ Unchanged View Storybook View Build 2026-01-16 05:45:18

@github-actions
Copy link

github-actions bot commented Jan 16, 2026

🚀 PR Preview Report

Build가 성공적으로 완료되었습니다.

Preview에서 변경사항을 확인하세요.

Status Preview Build Log Updated (UTC)
✅ Ready Visit Preview View Logs 2026-01-16 05:45:56

@github-actions
Copy link

github-actions bot commented Jan 16, 2026

📊 Coverage Report

Status Build Log Updated (UTC)
✅ Ready View Build 2026-01-16 05:45:01

📉 #379main에 병합하면 coverage가 0.09% 감소합니다.

Coverage 요약

@@             Coverage Diff             @@
##             main     #379       +/-   ##
===========================================
- Coverage   35.57%   35.48%    -0.09%     
===========================================
  Files         254      254         0     
  Lines       12017    12053       +36     
  Branches      477      477         0     
===========================================
+ Hits         4275     4277        +2     
+ Misses       7742     7776       +34     

영향받은 파일

파일 Coverage 변화
/home/runner/work/WeGo_FrontEnd/WeGo_FrontEnd/src/hooks/use-notification/use-notification-connect-sse/index.ts 8.90% (-1.10%) ⬇️
/home/runner/work/WeGo_FrontEnd/WeGo_FrontEnd/src/providers/provider-notification/index.tsx 48.78% (-12.51%) ⬇️

@Chiman2937 Chiman2937 added the Ready For Review! 리뷰 받을 준비가 되었습니다. label Jan 16, 2026
Copy link
Member

@HopeFullee HopeFullee left a comment

Choose a reason for hiding this comment

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

각 케이스별로 함수 분리해서 보는 눈이 즐겁네요!
수고하셨습니다.

@Chiman2937 Chiman2937 merged commit cee6d65 into main Jan 16, 2026
9 of 10 checks passed
@Chiman2937 Chiman2937 deleted the chiyoung-fix/sse branch January 16, 2026 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready For Review! 리뷰 받을 준비가 되었습니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants