Skip to content

refactor: 주요 flex wrapper를 side Flex로 마이그레이션 - #186

Merged
KYBee merged 7 commits into
developfrom
refact/flex-migration-summary-badge
Jun 9, 2026
Merged

refactor: 주요 flex wrapper를 side Flex로 마이그레이션#186
KYBee merged 7 commits into
developfrom
refact/flex-migration-summary-badge

Conversation

@KYBee

@KYBee KYBee commented May 29, 2026

Copy link
Copy Markdown
Contributor

Refactor: side Flex 마이그레이션을 위한 테스트 정리 및 1유형 wrapper 전환

작업 내용

  • RecruitmentSummary, Badge, ContactSection의 현재 렌더링 계약을 테스트로 먼저 고정했습니다.
  • 단순 flex wrapper를 @sipe-team/sideFlex로 전환했습니다.
  • 마이그레이션 후 과한 테스트를 정리해 필요한 검증만 남겼습니다.

변경 사항

  • 테스트 추가 및 정리
    • RecruitmentSummary 상태별 렌더링 테스트 추가
    • Badge 텍스트 렌더링 테스트 추가
    • ContactSection 문의 문구와 이메일 링크 렌더링 테스트 추가
    • 마이그레이션 후 구현 세부사항에 가까운 검증은 정리
  • 1유형 마이그레이션 적용
    • RecruitmentSummary timer wrapper
    • Badge inline flex wrapper
    • ContactSection wrapper 및 email link wrapper
  • 기존 SCSS에서 중복된 flex wrapper 선언 정리
  • 반응형 flex 값은 현재 Flex API 범위 밖이라 SCSS media query에 유지

확인 사항

  • Node 20 기준 yarn test 통과 (9 files / 16 tests)
  • 대상 파일 ESLint 통과
  • 대상 파일 Prettier check 통과
  • git diff --check develop..HEAD 통과

@vercel

vercel Bot commented May 29, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sipe-team Ready Ready Preview, Comment May 30, 2026 9:55am

@KYBee KYBee changed the title refactor: RecruitmentSummary와 Badge Flex 마이그레이션 refactor: 주요 flex wrapper를 side Flex로 마이그레이션 May 30, 2026
@KYBee
KYBee marked this pull request as ready for review May 31, 2026 08:09
@KYBee
KYBee requested a review from G-hoon as a code owner May 31, 2026 08:09

@G-hoon G-hoon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

넵 수고하셨습니다.
추후에 토큰화된 디자인 시스템 적용 시에,

<Flex
...
gap="12px"

이런 값들은 통일되면 좋을 것 같습니다!

@KYBee

KYBee commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

@G-hoon 넵 추후에 토큰 적용까지 해보려 합니다~ 감사합니다

@KYBee
KYBee merged commit 92d192f into develop Jun 9, 2026
4 checks passed
@KYBee KYBee self-assigned this Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants