Skip to content

Conversation

@gdaegeun539
Copy link
Member

@gdaegeun539 gdaegeun539 commented Jan 5, 2026

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines

What kind of change does this PR introduce?

Select your one and delete others

  • Add feature

What is the current behavior?

You can also link to an open issue here

SCRUM-256, SCRUM-257, SCRUM-260 등의 내부 로그인 및 회원가입 로직이 구현되지 않은 상황입니다.

What is the new behavior (if this is a feature change)?

  • 로그인 및 회원가입에서의 내비게이션 흐름 구현

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

확인중입니다.

ScreenShots (If needed)

Other information:

화면 흐름만 구성한 상태입니다.

Implement navigation from login screen through terms agreement
to gender/age selection and finally to home screen.

- Add OnboardingTerms and OnboardingGenderAge destinations
- Add navigation callbacks to LoginScreen
- Add navigation callbacks to OnboardingTermsScreen
- Add navigation callbacks to OnboardingGenderAgeScreen
- Enable non-login entry button
@gdaegeun539 gdaegeun539 self-assigned this Jan 5, 2026
@gdaegeun539 gdaegeun539 added the feat 새로운 기능 label Jan 5, 2026
@gdaegeun539 gdaegeun539 marked this pull request as draft January 5, 2026 09:55
Copy link
Collaborator

@hyunjung-choi hyunjung-choi left a comment

Choose a reason for hiding this comment

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

감사합니다 ! 🥲

@gdaegeun539 gdaegeun539 changed the title SCRUM-??? feat: configure signup flow SCRUM-256 SCRUM-257 feature: configure signup flow Jan 9, 2026
@gdaegeun539
Copy link
Member Author

@hyunjung-choi 여기에 SCRUM-260까지 구현하는 게 좋을까요...?

@hyunjung-choi
Copy link
Collaborator

아뇨 따로 빼는 게 좋을 것 같아요 !!

@gdaegeun539
Copy link
Member Author

아뇨 따로 빼는 게 좋을 것 같아요 !!

그러면 코드 보면서 문제 있을지 확인해보고 괜찮으면 Draft 해제할게요.

@gdaegeun539
Copy link
Member Author

코드가 생각보다 용납할 만 하네요? 드래프트 풀겠습니다

  • 여담

이젠 토르발즈 아저씨도 비 코어 로직에는 바이빙한다고 하니까 당연한 걸지도요...

@gdaegeun539 gdaegeun539 marked this pull request as ready for review January 11, 2026 15:04
@gdaegeun539
Copy link
Member Author

별 문제 없는 것으로 확인하고 병합하겠습니당

@gdaegeun539 gdaegeun539 merged commit d0af08e into project-lyrics:develop Jan 16, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 새로운 기능

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants