Skip to content

Refactor/#474 3-2차 스프린트 QA 반영 - #475

Merged
dev-domo merged 2 commits into
developfrom
refactor/#474-seungjun-qa-3-2
Jul 20, 2026
Merged

Refactor/#474 3-2차 스프린트 QA 반영#475
dev-domo merged 2 commits into
developfrom
refactor/#474-seungjun-qa-3-2

Conversation

@dev-domo

@dev-domo dev-domo commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

🔗 연결된 이슈

📄 작업 내용

  • 공통퀘스트 답변 상세 보기에서 뒤로 가기 시 나의 여정 탭으로 떨어지는 오류가 있어 해결했습니다.
  • 알림 탭에서 헤더 뷰(최근 30일까지의 알림 표시 + 모두 읽기 버튼)과 첫 알림 간의 간격을 조정했습니다.

Summary by CodeRabbit

  • 개선 사항
    • 공지 및 알림 목록의 섹션 간 여백과 헤더 표시를 조정해 화면 구성이 더 자연스러워졌습니다.
    • 퀘스트 화면을 다시 열 때 이전에 선택한 탭과 콘텐츠가 유지되도록 개선했습니다.

@dev-domo dev-domo self-assigned this Jul 17, 2026
@dev-domo dev-domo added 승준🤠 refactor 기존 코드 리팩토링 labels Jul 17, 2026
@dev-domo dev-domo linked an issue Jul 17, 2026 that may be closed by this pull request
3 tasks
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

홈 알림 화면의 섹션 헤더 여백과 높이를 조정하고, 퀘스트 화면 재진입 시 선택된 하위 화면과 탭 상태를 유지하도록 변경했습니다.

Changes

홈 알림 헤더 표시

Layer / File(s) Summary
알림 헤더 레이아웃
ByeBoo-iOS/.../NoticeCardsView.swift, ByeBoo-iOS/.../NotificationsViewController.swift
알림 카드의 sectionHeaderTopPadding을 0으로 설정하고, 알림 목록은 첫 섹션을 제외한 헤더에 24.adjustedH 높이를 적용합니다.

퀘스트 선택 상태 표시

Layer / File(s) Summary
선택된 퀘스트 화면 복원
ByeBoo-iOS/.../ParentQuestViewController.swift
viewWillAppear에서 selectedIndex에 해당하는 컨트롤러와 탭을 표시하며, viewWillDisappear의 인덱스 초기화 로직을 제거합니다.

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

Possibly related PRs

Suggested reviewers: juri123123, y-eonee

Poem

깡충 토끼 화면을 보니
헤더 여백 가지런해졌네
선택한 퀘스트 다시 만나
탭도 함께 반짝반짝
당근처럼 상태가 딱 맞네 🐰

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive 제목이 QA 반영이라는 큰 틀만 보여주고 실제 변경점인 알림 헤더 간격과 퀘스트 탭 복귀 수정이 드러나지 않습니다. 알림 헤더 간격 조정과 퀘스트 상세 복귀 시 탭 동작 수정처럼 핵심 변경을 드러내는 구체적인 제목으로 바꾸세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ 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 refactor/#474-seungjun-qa-3-2

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.

@y-eonee y-eonee left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

고생하셨습니다 !!

@dev-domo
dev-domo merged commit b5f97fb into develop Jul 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 기존 코드 리팩토링 승준🤠

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Refactor] 3-2차 스프린트 QA 반영

3 participants