Skip to content

feature#52 장바구니 Redirect 구현 #63

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 30 commits into from
Oct 27, 2025
Merged

feature#52 장바구니 Redirect 구현 #63

merged 30 commits into from
Oct 27, 2025

Conversation

@soochan8
Copy link
Collaborator

@soochan8 soochan8 commented Oct 21, 2025 •
edited
Loading

📌 Issue

🔖 요약

  1. 장바구니 redirect 설정
  2. 공통 세션 관리 추가

🛠 작업 내용

  1. SessionState, CheckSessionUseCase
  • SessionState를 추가하여, 각 Feature 모듈에서 사용할 수 있도록 구현하였습니다.
  • 세션확인 UseCase 추가하였습니다.
  1. CartContract
  • 공통 SessionState를 override하여 세션관리 상태를 추가하였습니다.
  1. CartNavGraph
  • 세션이 없을 때(로그인이 안되어 있다고 가정), 장바구니 화면 진입 및 상품을 장바구니에 추가하려고 할 때, 로그인 화면으로 이동 후 Redirect 경로를 설정하였습니다.
  • Cart 진입과 상품 클릭(CartPopup)의 각 Redirect 경로를 설정하였습니다.

@soochan8 soochan8 requested a review from f-lab-pepe October 21, 2025 04:47
@soochan8 soochan8 self-assigned this Oct 21, 2025
@soochan8 soochan8 added the feat 새로운 기능 추가 label Oct 21, 2025
@soochan8 soochan8 changed the base branch from main to develop October 21, 2025 04:47
f-lab-pepe
f-lab-pepe reviewed Oct 21, 2025
View reviewed changes
@soochan8 soochan8 requested a review from f-lab-pepe October 22, 2025 06:49
f-lab-pepe
f-lab-pepe approved these changes Oct 23, 2025
View reviewed changes
Copy link
Collaborator

@f-lab-pepe f-lab-pepe left a comment

Choose a reason for hiding this comment

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

LGTM!

@soochan8 soochan8 merged commit 0e65343 into develop Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@f-lab-pepe f-lab-pepe f-lab-pepe approved these changes

Assignees

@soochan8 soochan8

Labels

feat 새로운 기능 추가

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@soochan8 @f-lab-pepe