Skip to content

Conversation

@HopeFullee
Copy link
Member

📝 변경 사항

[수확]

Tanstack-Form의 form meta 상태 중 "isSubmitting"은 onSubmit에 실행 하는 useRouter의 실행은 보장되지만,
router의 경로 이동이 끝나는 시점까진 보장되지 않음. 즉, isSubmitting을 기준으로 submit button의 disabled 속성을
제어하는 것은 추천하지 않고 대신 router의 경로 이동까지 보장되는 "isSubmitted"를 추천함.


🔗 관련 이슈

Closes #


🧪 테스트 방법

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

📸 스크린샷 (선택)


📋 체크리스트

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

💬 추가 코멘트


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

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

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 19, 2025

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.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch somang-feat/post-meetup

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
Copy link

github-actions bot commented Dec 19, 2025

🎨 Storybook Report

변경 사항이 없습니다

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

Status Storybook Build Log Updated (UTC)
✅ Unchanged View Storybook View Build 2025-12-19 04:07:26

@github-actions
Copy link

github-actions bot commented Dec 19, 2025

📊 Coverage Report

Status Build Log Updated (UTC)
✅ Ready View Build 2025-12-19 04:07:00

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

Coverage 요약

@@             Coverage Diff             @@
##             main     #212       +/-   ##
===========================================
- Coverage   33.66%   33.62%    -0.04%     
===========================================
  Files         155      155         0     
  Lines        6538     6545        +7     
  Branches      249      249         0     
===========================================
  Hits         2201     2201         0     
+ Misses       4337     4344        +7     

영향받은 파일

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

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

@github-actions
Copy link

github-actions bot commented Dec 19, 2025

🚀 PR Preview Report

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

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

Status Preview Build Log Updated (UTC)
✅ Ready Visit Preview View Logs 2025-12-19 04:08:10

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

github-actions bot commented Dec 19, 2025

🎭 Playwright Report

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

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

Status Build Log Updated (UTC)
✅ Ready View Build 2025-12-19 04:08:44

📊 Test Summary

  • ✅ Passed: 9
  • ❌ Failed: 0
  • ⏱️ Duration: 46.1s

📜 Test Details

✅ Passed Tests (9)
  • profile.test.ts (9)
    • [chromium] 비로그인 상태에서 /mypage 접속 시 /login으로 redirect 되는 지 테스트
    • [chromium] 나의 프로필 페이지로 접속 시 /mypage로 /redirect 되는 지 테스트
    • [chromium] 존재하지 않는 프로필 페이지로 접속 시 404 redirect 되는 지 테스트
    • [firefox] 비로그인 상태에서 /mypage 접속 시 /login으로 redirect 되는 지 테스트
    • [firefox] 나의 프로필 페이지로 접속 시 /mypage로 /redirect 되는 지 테스트
    • [firefox] 존재하지 않는 프로필 페이지로 접속 시 404 redirect 되는 지 테스트
    • [webkit] 비로그인 상태에서 /mypage 접속 시 /login으로 redirect 되는 지 테스트
    • [webkit] 나의 프로필 페이지로 접속 시 /mypage로 /redirect 되는 지 테스트
    • [webkit] 존재하지 않는 프로필 페이지로 접속 시 404 redirect 되는 지 테스트

</section>

<MeetupSubmitButton onSubmitClick={() => form.handleSubmit()} />
<form.Subscribe
Copy link
Member

Choose a reason for hiding this comment

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

subscribe
미국∙영국 [səbˈskraɪb]
영국식

동사
1
(신문 등을) 구독하다, (인터넷·유료 TV 채널 등에[을]) 가입[시청]하다
Which journals does the library subscribe to?

이 도서관에서는 어떤 저널들을 구독하나요?
2
(단체에 유료 회원으로) 가입하다, (자선 단체에 정기적으로) 기부하다
He subscribes regularly to Amnesty International.

그는 국제 사면 위원회에 정기적으로 기부를 한다.
3
(주식을 사기 위해) 청약[신청]하다 (→oversubscribed)

정답:

Copy link
Member Author

@HopeFullee HopeFullee Dec 19, 2025

Choose a reason for hiding this comment

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

정답: 1번

  • <form.Subscribe /> 태그에서부터 알수 있듯이 form 에 구독을 해서 form 의 meta 정보 ("isSubmitting" 등..) 를 실시간으로 받아옴.
  • Subscribe의 특이점: 구독을 통해 form의 meta 정보를 얻어올땐 form 자체는 re-render가 발생하지 않고 오로지 구독을 하고 있는 children만 re-render를 함.

Copy link
Member

Choose a reason for hiding this comment

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

@HopeFullee HopeFullee merged commit 14e0056 into main Dec 19, 2025
11 checks passed
@HopeFullee HopeFullee deleted the somang-feat/post-meetup branch December 19, 2025 04:23
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