Skip to content

Conversation

@HopeFullee
Copy link
Member

@HopeFullee HopeFullee commented Dec 19, 2025

📝 변경 사항

[삽질]

  • onChange 버리고 onBlur 적용
  • onBlur만 하니깐 이상해 보여서 onChange 쓰까봄
  • tanstack-form이 onBlur + onChange를 동시에 지원안함. (onChange 또는 onBlur 둘중에 발동하는 한가지가 선점을 함.)
  • onChange를 다시보니 선녀, 원래 상태로 원복
  • 잃어 버린 4시간

[그나마 한거]

  • 제목 필드 2자 내외 경고를 빼버림. 이유는 어차피 서버에서도 1자 이상부터 받고 있기도 했고.
    2자 내외를 걸어버리면, onChange로 발생하는 경고 메세지가 글자 단 1개만 써도 즉시 튀어나오는 불쾌함이 있음.
    그래서 그냥 nonEmpty만 거는걸로 셀프 합의봄 (자기 합리화).
  • 상세 필드 300자 제한을 textArea의 max 속성으로 걸어놨다고 귀찮아서 zod 경고 안함.
    하지만 본인같은 악질 유저들은 개발자 도구 키고 max값을 수정할 것을 알기에 zod로 300자 이내 제한함.
    (물론 서버에서 막히겠지만 API 요청 자체를 원천 차단)
  • 태그 필드 입력시 배열에 추가되는 형태이기 때문에 zod를 통해 유효성 검사하기가 까다로움.
    그래서 Enter로 입력 받는 시점에서 (기존 배열에 중복 태그는 없는지, 길이는 1~8자 인지, 총 개수가 10개를 넘지 않는지)
    하드코딩 검증을 통해 우효~~한 태그만 입력받음.

🔗 관련 이슈

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.

@HopeFullee HopeFullee added the Ready For Review! 리뷰 받을 준비가 되었습니다. label Dec 19, 2025
@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 11:13:44

@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 11:14:19

@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 11:15:06

📊 Test Summary

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

📜 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 되는 지 테스트

@github-actions
Copy link

github-actions bot commented Dec 19, 2025

📊 Coverage Report

Status Build Log Updated (UTC)
✅ Ready View Build 2025-12-19 11:13:26

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

Coverage 요약

@@             Coverage Diff             @@
##             main     #214       +/-   ##
===========================================
- Coverage   33.68%   33.66%    -0.02%     
===========================================
  Files         155      155         0     
  Lines        6551     6555        +4     
  Branches      250      250         0     
===========================================
  Hits         2207     2207         0     
+ Misses       4344     4348        +4     

영향받은 파일

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

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

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

다음엔 우효~~~~~~~~한 검사로 해주세요.

우효~~!! 겟☆또Daze~ 초Lucky☆다~!

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