Skip to content

온보딩 페이지와 관련 에셋, 라이브러리 제거#24

Merged
yoouyeon merged 1 commit intodevelopfrom
refactor/MD-9
Feb 22, 2026
Merged

온보딩 페이지와 관련 에셋, 라이브러리 제거#24
yoouyeon merged 1 commit intodevelopfrom
refactor/MD-9

Conversation

@yoouyeon
Copy link
Contributor

@yoouyeon yoouyeon commented Feb 19, 2026

#23 pr에 이어서 작업한거라 base 브랜치를 우선 해당 브랜치 쪽으로 해서 pr 만들었습니다!
머지하기 전에 base 브랜치를 develop으로 바꾸겠습니다.

💻 작업 내용

  • 필요성 부족으로 인해 제거하기로 결정한 온보딩 페이지를 제거했습니다.
  • 온보딩 페이지에서만 사용하던 에셋과 swiper 라이브러리 제거했습니다.
  • 로그인 이후 온보딩 페이지가 아닌 홈 페이지로 이동하도록 변경했습니다.

Summary by CodeRabbit

변경 사항

  • 제거된 기능

    • 온보딩 페이지 및 관련 기능 완전 제거
    • 온보딩 UI 컴포넌트와 애니메이션 삭제
  • 개선 사항

    • 게스트 로그인 시 온보딩 과정을 건너뛰고 홈 화면으로 즉시 이동

- 필요성 부족으로 인해 제거하기로 결정한 온보딩 페이지를 제거
- 온보딩 페이지에서만 사용하던 에셋과 swiper 라이브러리 제거
- 로그인 이후 온보딩 페이지가 아닌 홈 페이지로 이동하도록 변경
@yoouyeon yoouyeon self-assigned this Feb 19, 2026
@yoouyeon yoouyeon added the 🔨 Refactor 코드 구조 개선 label Feb 19, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 19, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

둘러보기

온보딩 기능 전체가 제거되었습니다. 이에 따라 Swiper 의존성도 삭제되고, 라우팅에서 온보딩 페이지가 제외되며, 로그인 후 사용자는 홈 페이지로 직접 이동하게 됩니다.

변경 사항

코호트 / 파일 요약
의존성 제거
package.json, src/vite-env.d.ts
Swiper 라이브러리 및 타입 정의(@types/swiper) 제거, Swiper CSS 모듈 선언 제거
라우팅 및 네비게이션 업데이트
src/app/Router.tsx, src/pages/login/LoginPage.tsx, src/shared/config/route.ts
LazyOnboarding 라우트 항목 제거, 로그인 후 네비게이션 대상을 온보딩에서 홈으로 변경, ROUTE 객체에서 'onboarding' 경로 제거
온보딩 페이지 제거
src/pages/onboarding/OnboardingPage.tsx, src/pages/onboarding/OnboardingPage.style.ts, src/pages/onboarding/index.ts
OnboardingPage 컴포넌트 전체 파일 삭제(137줄), 스타일 정의 완전 제거(102줄), 모듈 익스포트 제거
공유 UI 컴포넌트 정리
src/shared/ui/Lottie/index.tsx, src/shared/ui/Lottie/index.styles.ts
4개의 온보딩 단계별 Lottie 컴포넌트 제거, 온보딩 관련 Lottie JSON 임포트 제거, ExtendedWrapper 및 OnboardingWrapper 스타일 제거

🐰 온보딩은 작별을 고하고,
Swiper는 사라지고,
홈으로 곧장 가는 길이 열렸네.
단순함이 우리를 반긴다.
설레는 다음 장으로!

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/MD-9

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 Feb 19, 2026

📝 관련 이슈

MD-9

@github-actions github-actions bot requested a review from ongheong February 19, 2026 01:05
@yoouyeon
Copy link
Contributor Author

@coderabbitai full review

@coderabbitai
Copy link

coderabbitai bot commented Feb 19, 2026

✅ Actions performed

Full review triggered.

Copy link
Contributor

@ongheong ongheong left a comment

Choose a reason for hiding this comment

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

👍🏻👍🏻👍🏻

Base automatically changed from feat/MD-5 to develop February 22, 2026 13:41
@yoouyeon yoouyeon merged commit bb38e5c into develop Feb 22, 2026
3 checks passed
@yoouyeon yoouyeon deleted the refactor/MD-9 branch February 22, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 Refactor 코드 구조 개선

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants