Skip to content

fix: 온보딩 운동 질문의 답변 목록이 서로 바뀐 문제 수정 - #208

Merged
JioCoder merged 1 commit into
mainfrom
fix/onboarding-exercise-options-swap
Sep 1, 2026
Merged

fix: 온보딩 운동 질문의 답변 목록이 서로 바뀐 문제 수정#208
JioCoder merged 1 commit into
mainfrom
fix/onboarding-exercise-options-swap

Conversation

@JioCoder

@JioCoder JioCoder commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

"운동 시간은 어떻게 되시나요?"에 빈도 보기(거의 매일, 주 34회…)가,
"운동 빈도는 어떻게 되시나요?"에 시간 보기(4시간 이상, 2
4시간…)가
나오고 있었다. 저장되는 값도 반대라 exerciseDuration에 빈도가,
exerciseFrequency에 시간이 들어갔다.

각 섹션이 자기 질문에 맞는 보기와 상태를 쓰도록 바로잡는다.
섹션 노출 조건도 함께 맞춘다 — 항상 보이는 빈도 섹션을 고르면
시간 섹션이 위에 나타난다(기존 동작 유지).

버전을 1.2.0으로 올린다.

Summary by CodeRabbit

  • 새 기능

    • 온보딩 운동 설정에서 운동 빈도와 운동 시간 선택 항목이 올바르게 표시되고 매핑됩니다.
    • 운동 빈도는 항상 표시되며, 운동 시간은 운동 빈도 선택 후 표시됩니다.
  • 기타

    • 앱 버전이 1.2.0으로 업데이트되었습니다.

"운동 시간은 어떻게 되시나요?"에 빈도 보기(거의 매일, 주 3~4회…)가,
"운동 빈도는 어떻게 되시나요?"에 시간 보기(4시간 이상, 2~4시간…)가
나오고 있었다. 저장되는 값도 반대라 exerciseDuration에 빈도가,
exerciseFrequency에 시간이 들어갔다.

각 섹션이 자기 질문에 맞는 보기와 상태를 쓰도록 바로잡는다.
섹션 노출 조건도 함께 맞춘다 — 항상 보이는 빈도 섹션을 고르면
시간 섹션이 위에 나타난다(기존 동작 유지).

버전을 1.2.0으로 올린다.
@coderabbitai

coderabbitai Bot commented Sep 1, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 88ee291b-cca5-4e5e-910b-deed0ad409d0

📥 Commits

Reviewing files that changed from the base of the PR and between e7d082c and e4e41eb.

📒 Files selected for processing (2)
  • app.config.js
  • features/onboarding/components/onboarding-exercise-detail-screen.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

온보딩 화면에서 운동 빈도와 운동 시간 섹션의 표시 조건과 옵션 매핑을 수정했습니다. 앱 버전을 1.2.0으로 갱신했습니다.

Changes

온보딩 및 릴리스 갱신

Layer / File(s) Summary
운동 선택 섹션 매핑 수정
features/onboarding/components/onboarding-exercise-detail-screen.tsx
운동 빈도 섹션을 항상 표시합니다. 운동 빈도 선택 후 운동 시간 섹션을 표시하고 DURATION_OPTIONS를 렌더링합니다.
앱 버전 갱신
app.config.js
앱 버전을 1.1.1에서 1.2.0으로 변경했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to e4e41

The PR corrects the exercise duration and frequency options so each is saved through its matching field while preserving the existing onboarding flow, and updates the app version metadata. No actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: casebread

🚥 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%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 온보딩 운동 질문의 운동 시간과 운동 빈도 답변 목록이 서로 바뀐 문제를 수정한다는 주요 변경 사항을 정확하게 설명합니다.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/onboarding-exercise-options-swap

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.

@JioCoder
JioCoder merged commit 8e91af1 into main Sep 1, 2026
3 checks passed
@JioCoder
JioCoder deleted the fix/onboarding-exercise-options-swap branch September 1, 2026 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant