Skip to content

Conversation

@MoonsuKang
Copy link
Member

@MoonsuKang MoonsuKang commented Feb 21, 2025

Related issue 🛠

closed #167

어떤 변경사항이 있었나요?

  • 🐞 BugFix Something isn't working
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 컨벤션에 맞게 작성했습니다. (필수)
  • merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
  • BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

  • 리뷰 반영을 통해 AlarmReceiver에서 firstFortuneId 처리 수행

  • 운세 확인 시 첫 알람 여부에 따른 보상 분기 처리

  • FortuneViewModel에서 firstDismissedAlarmId를 구독하여 hasReward 결정

  • FortuneCompletePage에서 hasReward 값에 따라 버튼 및 메시지 변경

  • onCompleteClick에서 hasReward에 따라 NextStep 또는 NavigateToHome 호출

Uncompleted Tasks 😅

  • Task1

To Reviewers 📢

Copy link
Member

@DongChyeon DongChyeon left a comment

Choose a reason for hiding this comment

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

굿굿

@MoonsuKang MoonsuKang merged commit 5a3790f into develop Feb 21, 2025
1 check passed
@DongChyeon DongChyeon deleted the feat/#167-fortune-branch branch June 10, 2025 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 운세 보상에 대한 분기처리를 진행합니다.

3 participants