Skip to content

[Fix] #87 스크롤 화면 밀림 및 모바일 토스트메세지 버그 수정 - #88

Merged
hdg0116 merged 1 commit into
developfrom
fix/#87-오류사항-해결
May 31, 2026

Hidden character warning

The head ref may contain hidden characters: "fix/#87-\uc624\ub958\uc0ac\ud56d-\ud574\uacb0"
Merged

hdg0116 merged 1 commit into
developfrom
fix/#87-오류사항-해결

Conversation

@hdg0116

@hdg0116 hdg0116 commented May 31, 2026

Copy link
Copy Markdown
Collaborator

🗒️ PR 타입

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 코드 리팩토링
  • 문서 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

🔗 관련 이슈


📌 작업사항

  • 모바일 환경 토스트메세지 안 뜨는 버그 수정
  • 스크롤 생김 시 화면 밀림 제거

📸 스크린샷

추후첨부!!!!


📣 기타사항 및 코멘트


✅ 체크리스트

  • 로컬에서 빌드 및 테스트 완료
  • Label 지정 완료
  • 리뷰어 지정 완료
  • 코드 리뷰 반영 완료

Summary by CodeRabbit

Bug Fixes

  • 온보딩 레이아웃의 화면 오버플로우 처리 및 스크롤 동작 개선
  • 로그인 콜백 페이지의 에러 처리 강화로 에러 메시지 보존 및 페이지 이동 안정성 향상

@hdg0116
hdg0116 requested a review from nabbang6 May 31, 2026 02:18
@hdg0116 hdg0116 self-assigned this May 31, 2026
@hdg0116 hdg0116 added the 🐞 Fix 버그 해결 label May 31, 2026
@vercel

vercel Bot commented May 31, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
bangjjack-frontend-dev Ready Ready Preview, Comment May 31, 2026 2:19am

@coderabbitai

coderabbitai Bot commented May 31, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

이 PR은 Onboarding 레이아웃의 스크롤 오버플로우 문제를 해결하고, 로그인 토큰 교환 실패 시 에러 메시지를 sessionStorage에 저장하는 두 가지 개선사항을 포함합니다.

Changes

Onboarding 레이아웃 스크롤 개선

Layer / File(s) Summary
레이아웃의 스크롤 및 오버플로우 처리
src/features/onboarding/components/OnBoardingLayout.tsx
최상위 컨테이너를 min-h-dvh에서 h-dvh로 변경하고 overflow-hidden을 추가하여 고정 높이를 설정합니다. 본문 영역(main)에 overflow-y-auto, scrollbar-none, pb-800을 적용하여 본문만 스크롤 가능하도록 조정합니다.

로그인 에러 처리 개선

Layer / File(s) Summary
토큰 교환 실패 시 에러 메시지 저장
src/pages/LoginCallbackPage.tsx
LOGIN_ERROR_MESSAGE_KEY를 import하여 토큰 교환 실패 시 sessionStorage에 에러 메시지를 저장한 뒤 로그인 페이지로 리다이렉트합니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Suggested reviewers

  • nabbang6
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 이슈 #87과 연관되어 있으며, 스크롤 화면 밀림 및 모바일 토스트메세지 버그 수정이라는 주요 변경사항을 명확하게 요약하고 있습니다.
Linked Issues check ✅ Passed 코드 변경사항은 이슈 #87의 두 가지 요구사항인 스크롤 화면 밀림 제거(OnBoardingLayout 수정)와 모바일 토스트메세지 버그 수정(LoginCallbackPage 수정)을 모두 충족합니다.
Out of Scope Changes check ✅ Passed 두 개의 파일 수정(OnBoardingLayout.tsx, LoginCallbackPage.tsx) 모두 이슈 #87의 요구사항 범위 내에서 진행되었으며, 범위 외의 변경사항은 없습니다.
Description check ✅ Passed PR 설명이 기본 구조는 갖추었으나 스크린샷 미첨부, 기타사항 공란, 코드 리뷰 반영 미완료 상태입니다.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/#87-오류사항-해결

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

✅ CI 검증 결과

TypeScript: 통과
ESLint: 통과
Prettier: 통과
Build: 통과

🎉 모든 검증을 통과했습니다.

@hdg0116
hdg0116 merged commit f30560a into develop May 31, 2026
4 checks passed
@hdg0116
hdg0116 deleted the fix/#87-오류사항-해결 branch May 31, 2026 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 Fix 버그 해결

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] #87 오류사항 해결

1 participant