Skip to content

Comments

프로필 수정 타이틀 수정#29

Merged
Sangyoon98 merged 2 commits intomainfrom
dev
Nov 9, 2025
Merged

프로필 수정 타이틀 수정#29
Sangyoon98 merged 2 commits intomainfrom
dev

Conversation

@Sangyoon98
Copy link
Member

@Sangyoon98 Sangyoon98 commented Nov 9, 2025

📝 Summary

프로필 수정 타이틀 수정

🙏 Question & PR point

📬 Reference

Summary by CodeRabbit

릴리스 노트 (버전 4)

  • Chores

    • 프로젝트 버전 업데이트
  • 개선 사항

    • 프로필 업데이트 화면의 UI 개선으로 섹션 제목과 헤더 레이블 추가

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 9, 2025

Walkthrough

프로젝트 버전이 3에서 4로 상향되었으며, UpdateProfileBottomSheet에 헤더 레이블과 섹션 제목이 추가되어 프로필 수정 시트의 UI 레이아웃이 개선되었습니다.

Changes

Cohort / File(s) 변경 요약
프로젝트 버전 업데이트
SampoomManagement.xcodeproj/project.pbxproj
Debug와 Release 빌드 설정에서 CURRENT_PROJECT_VERSION을 3에서 4로 변경
프로필 수정 시트 UI 개선
SampoomManagement/Features/User/UI/UpdateProfileBottomSheet.swift
수직 스택 상단에 Spacer와 헤더 레이블 추가, "직급 선택" 섹션 제목 추가, 내부 VStack에서 navigationTitle 및 background 모디파이어 제거

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • 프로젝트 버전 번프는 자동 관리 사항
  • UI 레이아웃 변경은 비교적 단순한 SwiftUI 모디파이어 추가/제거

Poem

🐰 프로필 시트가 더욱 예뻐졌네요,
헤더와 제목이 반갑게 인사를 하고,
버전도 쏘옥~ 4로 올라갔어요.
작은 변화지만 큰 마음으로 축하합니다! 🎉

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 실제 변경 사항과 정확하게 일치합니다. 프로필 수정 화면의 타이틀/헤더를 추가하고 수정하는 것이 주요 변경 사항이며, 제목이 이를 명확하게 반영합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

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 and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8fb94d4 and 61f5736.

📒 Files selected for processing (2)
  • SampoomManagement.xcodeproj/project.pbxproj (2 hunks)
  • SampoomManagement/Features/User/UI/UpdateProfileBottomSheet.swift (1 hunks)
🔇 Additional comments (2)
SampoomManagement.xcodeproj/project.pbxproj (1)

287-287: 버전 증가가 올바르게 적용되었습니다.

Debug 및 Release 구성 모두에서 프로젝트 버전이 3에서 4로 일관되게 업데이트되었습니다.

Also applies to: 322-322

SampoomManagement/Features/User/UI/UpdateProfileBottomSheet.swift (1)

35-40: 커스텀 헤더 구현이 적절합니다.

StringResources.Setting.editProfile을 사용한 커스텀 헤더 구현이 올바르게 되어 있습니다. 폰트, 색상, 정렬 및 패딩이 적절하게 적용되었습니다.

Copy link

@Lee-Jong-Jin Lee-Jong-Jin left a comment

Choose a reason for hiding this comment

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

수정

@Sangyoon98 Sangyoon98 merged commit 9fe726e into main Nov 9, 2025
6 checks passed
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.

2 participants