Skip to content

Conversation

@HopeFullee
Copy link
Member

📝 변경 사항

[문제]

  • 기존 모임 삭제 이후 목록으로 replace -> 이동 이후 기존 목록이 캐싱되어 방금 삭제한 모임이 그대로 보임

[해결]

  • 모임 삭제 이후 모임 목록 쿼리 키 invalidate로 refetch 적용

[트러블슈팅]

  • 모임 삭제 이후 history back을 통해 방금 삭제했던 모임의 상세 페이지로 이동할 수 있음. (replace 를 적용했다 하더라도 악의 적으로 history를 2~3번 이상 쌓으면 replace 또한 해결책이 될 수 없음).
  • history back으로 삭제된 모임 상세 페이지내에서 특별히 할 수 있는 것은 없음 (서버에서 막음 처리)

[트러블슈팅 삽질 과정]

  • tanstack-query의 caching이 문제라 생각해봄 그래서 모임 delete 이후 removeQueries를 적용해서 detail 캐싱을 날려버림.
    여전히 history back으로 상세 페이지에 접근하면 예전 정보가 그대로 보임.
  • 브라우저 자체도 history에 캐싱을 해둔다는 것을 알게됨 그래서 next/navigation의 useRouter.refresh 기능을 사용해 봤지만
    잘 못 사용한건지 여전히 history back 캐시를 날리지 못함.

[결론]

  • history back 까지 신경 써야하는걸까?
  • 나중에 시간나면 다시 돌아와 보겠음.

🔗 관련 이슈

Closes #


🧪 테스트 방법

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

📸 스크린샷 (선택)


📋 체크리스트

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

💬 추가 코멘트


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

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

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 1, 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.

@HopeFullee HopeFullee added the Ready For Review! 리뷰 받을 준비가 되었습니다. label Jan 1, 2026
@github-actions
Copy link

github-actions bot commented Jan 1, 2026

🎭 Playwright Report

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

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

Status Build Log Updated (UTC)
✅ Ready View Build 2026-01-01 13:24:03

📊 Test Summary

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

📜 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 1, 2026

🎨 Storybook Report

변경 사항이 없습니다

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

Status Storybook Build Log Updated (UTC)
✅ Unchanged View Storybook View Build 2026-01-01 13:23:18

@github-actions
Copy link

github-actions bot commented Jan 1, 2026

📊 Coverage Report

Status Build Log Updated (UTC)
✅ Ready View Build 2026-01-01 13:22:53

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

Coverage 요약

@@             Coverage Diff             @@
##             main     #309       +/-   ##
===========================================
- Coverage   39.99%   39.97%    -0.02%     
===========================================
  Files         213      213         0     
  Lines        9461     9467        +6     
  Branches      410      410         0     
===========================================
  Hits         3784     3784         0     
+ Misses       5677     5683        +6     

영향받은 파일

이 PR로 영향받은 파일이 없습니다

수정된 모든 파일이 현재 coverage를 유지했습니다.

@github-actions
Copy link

github-actions bot commented Jan 1, 2026

🚀 PR Preview Report

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

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

Status Preview Build Log Updated (UTC)
✅ Ready Visit Preview View Logs 2026-01-01 13:24:03

Copy link
Member

@claudia99503 claudia99503 left a comment

Choose a reason for hiding this comment

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

와우 이거 트러블 슈팅으로 적으시면 되겠네요 새해부터 고생 마이 하셨구만요
삭제된 페이지의 경우 서버에서 내려주는 에러 코드를 기준으로 해당 상세 페이지 진입 시 애초에 목록 페이지로 바로 리다이렉트 처리하는 방식은 어떨까요

@HopeFullee
Copy link
Member Author

와우 이거 트러블 슈팅으로 적으시면 되겠네요 새해부터 고생 마이 하셨구만요 삭제된 페이지의 경우 서버에서 내려주는 에러 코드를 기준으로 해당 상세 페이지 진입 시 애초에 목록 페이지로 바로 리다이렉트 처리하는 방식은 어떨까요

Cache에만 눈이멀어 있었는데, SSR 시점에서 prefetch할때 정보가 없으면 빠꾸 처리하는것도 좋은 방법이네요!
단지 여전히 킹받는건 history 자체에 캐싱을 강제적으로 해지할 수 있는가 없는가는 여전히 미스테리여서 나중에 공부좀 해보겠습니다.

@HopeFullee HopeFullee merged commit 2a8bae8 into main Jan 1, 2026
10 checks passed
@HopeFullee HopeFullee deleted the somang-fix/group-details branch January 1, 2026 14:41
@HopeFullee
Copy link
Member Author

방금 SSR 단에서 console 찍어봤는데, URL 눌러서 진입할때만 log 뜨고 history back 으로 진입할땐 SSR log가 안뜨는거 봐선 그냥 브라우저 history 캐싱 자체가 문제같습니다아아아아아아ㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏㅏ

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