[release] FE v1.8.9#1733
Merged
Merged
Conversation
React Compiler가 bail-out하는 Rules of React 위반을 도입 전 정리. - 조건부 훅: early return 가드를 모든 훅 호출 아래로 이동 (ApplicationFormPage, ClubCoverEditor, ClubLogoEditor, QuestionBuilder) - purity: 렌더 중 useRef(Date.now()) 제거 → effect/지연 초기화 (useTrackPageView, BuskingPage, IntroductionPage) - refs: 렌더 중 ref.current 읽기를 React Query isLoading으로 대체 (useGoogleCalendarData)
effect로 동기화하던 중복 상태를 렌더 중 파생값/지연 초기화로 정리. - ApplicantsTab: selectAll·isChecked를 checkedItem에서 파생 - ClubUnionPage: isMobile을 matchMedia로 지연 초기화, effect는 구독만
소스에서 제거된(Mixpanel 자동 distinct_id 연결) distinct_id 기대와 get_distinct_id 목/단언을 테스트에서 정리. 6건 실패 → 통과.
- babel-plugin-react-compiler@1.0.0을 @vitejs/plugin-react babel 옵션에 연결 (infer 모드). React 19라 런타임 패키지 불필요. - eslint-plugin-react-hooks@7 recommended-latest 연결, **/*.stories.tsx는 rules-of-hooks 오버라이드, set-state-in-effect는 정당한 효과/폼 시드 케이스가 섞여 warn 유지. 빌드(vite+tsc) 통과, 테스트 242개 통과.
- ApplicationFormPage: 가드를 훅 아래로 옮긴 뒤 params 부재 시 applicationAnswers_undefined_undefined 키로 쓰기되던 문제 수정. clubId·applicationFormId 둘 다 있을 때만 localStorage read/write. - ClubUnionPage: useState 지연 초기화/effect의 window.matchMedia를 존재 여부 가드로 감싸 JSDOM·구형 환경 런타임 크래시 방지.
스크롤 문제 때문에 다시 되돌려서 상태를 확인하고자함.
viewport 제거에 따른 헤더 위치가 이상한 문제 해결을 위한 것.
상단 헤더 고정이 되었던 시점인 거 같아서,,, 되돌립니다.
별도 이슈로 분리 작업 예정.
하단 safe-area 이슈로 분리
…indicator-MOA-1001 [feature] 실시간 순위 변동 인디케이터 추가
- ClickButton 140→160px - StartButton padding 10px 14px → 12px 18px - DropdownItem padding 10px → 12px - RankingBoard Item padding 10px 14px → 12px 16px
…wport-MOA-994 [fix] 좁은 뷰포트에서 sticky 탭 즉시 발동 버그 수정
…clip-MOA-996 [fix] PhotoModal 이미지 잘림 수정
…get-MOA-1002 [fix] 게임 페이지 모바일 클릭 영역 확대
- Modal: onClose를 ref로 관리하여 스크롤 복원 위치 깨짐 방지 - api.ts: window 가드 추가 (비브라우저 환경 ReferenceError 방지) - ClubUnionPage: window 존재 확인 후 matchMedia 호출 - DotTextEffect: 모듈 레벨 window.matchMedia에 가드 추가 - GamePage: body background 이전값 보존/복원 - close-issue-on-pr-merge.yml: action을 commit SHA로 고정 - docs: 코드펜스 언어 식별자 추가 (MD040)
3 tasks
[fix] PR #1733 코드리뷰 피드백 반영
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚀 릴리즈 PR
📦 버전 정보
💾 BE/💻 FE🚨 MAJOR/➕ MINOR/🔧 PATCHvX.Y.Z📖 버전 라벨 선택 가이드 (Semantic Versioning)
🚨 MAJORv1.0.0→v2.0.0➕ MINORv1.0.0→v1.1.0🔧 PATCHv1.0.0→v1.0.1📋 포함된 변경사항
Summary by CodeRabbit
릴리스 노트
New Features
Bug Fixes
Documentation