Skip to content

๐Ÿšธ Reset UX ๊ฐœ์„  (#234)#239

Merged
gaeng2y merged 1 commit into
developfrom
feature/#234-reset-ux
May 28, 2026
Merged

๐Ÿšธ Reset UX ๊ฐœ์„  (#234)#239
gaeng2y merged 1 commit into
developfrom
feature/#234-reset-ux

Conversation

@gaeng2y

@gaeng2y gaeng2y commented May 26, 2026

Copy link
Copy Markdown
Owner

๐Ÿ“ Summary

์˜ค๋Š˜ ์ˆ˜๋ถ„ ๊ธฐ๋ก ์ดˆ๊ธฐํ™” ์•ก์…˜์˜ ์˜ค์กฐ์ž‘ ๊ฐ€๋Šฅ์„ฑ์„ ๋‚ฎ์ถ”๊ธฐ ์œ„ํ•ด Reset์„ ๋ณด์กฐ ์œ„ํ—˜ ์•ก์…˜์œผ๋กœ ๋‚ฎ์ถ”๊ณ , ์‹คํ–‰ ์ „ ํ™•์ธ ์•Œ๋ฆผ๊ณผ HealthKit ์‚ญ์ œ ๋ฒ”์œ„ ์•ˆ๋‚ด๋ฅผ ์ถ”๊ฐ€ํ–ˆ์Šต๋‹ˆ๋‹ค.

๐ŸŽฏ Type of Change

  • โœจ New feature (์ƒˆ๋กœ์šด ๊ธฐ๋Šฅ ์ถ”๊ฐ€)
  • ๐Ÿ› Bug fix (๋ฒ„๊ทธ ์ˆ˜์ •)
  • ๐Ÿ”ง Configuration (์„ค์ • ๋ณ€๊ฒฝ)
  • โ™ป๏ธ Refactoring (์ฝ”๋“œ ๋ฆฌํŒฉํ† ๋ง)
  • ๐Ÿ“ Documentation (๋ฌธ์„œ ์ˆ˜์ •)
  • ๐ŸŽจ UI/UX (๋””์ž์ธ ๋ณ€๊ฒฝ)
  • โšก Performance (์„ฑ๋Šฅ ๊ฐœ์„ )
  • ๐Ÿ›ก๏ธ Security (๋ณด์•ˆ ๊ฐ•ํ™”)
  • ๐Ÿงช Test (ํ…Œ์ŠคํŠธ ์ถ”๊ฐ€/์ˆ˜์ •)
  • ๐Ÿ”จ Build (๋นŒ๋“œ ์‹œ์Šคํ…œ ์ˆ˜์ •)

๐Ÿ”— Related Issues

๐Ÿ“‹ Changes Made

Added

  • ์˜ค๋Š˜ ๊ธฐ๋ก ์ดˆ๊ธฐํ™” ํ™•์ธ ์•Œ๋ฆผ์„ ์ถ”๊ฐ€ํ–ˆ์Šต๋‹ˆ๋‹ค.
  • Reset ์„ฑ๊ณต ์‹œ ์ตœ๊ทผ ๊ธฐ๋ก ๋˜๋Œ๋ฆฌ๊ธฐ ์ƒํƒœ์™€ ์˜ค๋ฅ˜๊ฐ€ ์ •๋ฆฌ๋˜๋Š” ViewModel ํ…Œ์ŠคํŠธ๋ฅผ ์ถ”๊ฐ€ํ–ˆ์Šต๋‹ˆ๋‹ค.
  • ์ดˆ๊ธฐํ™” ์‚ญ์ œ ๋ฒ”์œ„๋ฅผ ์„ค๋ช…ํ•˜๋Š” ๋กœ์ปฌ๋ผ์ด์ฆˆ ๋ฌธ๊ตฌ๋ฅผ ์ถ”๊ฐ€ํ–ˆ์Šต๋‹ˆ๋‹ค.

Changed

  • Reset ๋ฒ„ํŠผ์„ ๊ธฐ๋ณธ ๊ธฐ๋ก CTA์™€ ๊ฐ™์€ ์ฃผ ์•ก์…˜์—์„œ ๋ถ„๋ฆฌํ•˜๊ณ  ๋ณด์กฐ ์œ„ํ—˜ ์•ก์…˜์œผ๋กœ ๋‚ฎ์ท„์Šต๋‹ˆ๋‹ค.
  • Reset ๋ฌธ๊ตฌ๋ฅผ ์˜ค๋Š˜ ๋ฌผ๋ฆฌ๋ฏธ๊ฐ€ ๋งŒ๋“  HealthKit ๊ธฐ๋ก๋งŒ ์‚ญ์ œํ•œ๋‹ค๋Š” ์˜๋ฏธ๋กœ ๋ช…ํ™•ํžˆ ๋ฐ”๊ฟจ์Šต๋‹ˆ๋‹ค.
  • ์ˆ˜๋ถ„ ๊ธฐ๋ก ์ œํ’ˆ ์ŠคํŽ™์— ์˜ค๋Š˜ ๊ธฐ๋ก ์ดˆ๊ธฐํ™” ์ •์ฑ…์„ ๋ฐ˜์˜ํ–ˆ์Šต๋‹ˆ๋‹ค.

Fixed

  • Reset ์ดํ›„ ์ตœ๊ทผ ๊ธฐ๋ก ๋˜๋Œ๋ฆฌ๊ธฐ ์ƒํƒœ์™€ undo ์˜ค๋ฅ˜๊ฐ€ ๋‚จ์ง€ ์•Š๋„๋ก ํšŒ๊ท€ ํ…Œ์ŠคํŠธ๋กœ ๋ณด๊ฐ•ํ–ˆ์Šต๋‹ˆ๋‹ค.

Removed

  • ๊ธฐ๋ณธ ๊ธฐ๋ก ๋ฒ„ํŠผ๊ณผ Reset ๋ฒ„ํŠผ์„ ๊ฐ™์€ ์šฐ์„ ์ˆœ์œ„๋กœ ๋ฐฐ์น˜ํ•˜๋˜ ์•ก์…˜ ๊ทธ๋ฃน์„ ์ œ๊ฑฐํ–ˆ์Šต๋‹ˆ๋‹ค.

๐Ÿ” Technical Details

  • DrinkWaterView์—์„œ ๊ธฐ๋ณธ ๊ธฐ๋ก CTA, ์ตœ๊ทผ ๊ธฐ๋ก ๋˜๋Œ๋ฆฌ๊ธฐ ์นด๋“œ, Reset ์•ก์…˜์„ ์„ธ๋กœ ํ๋ฆ„์œผ๋กœ ๋ถ„๋ฆฌํ–ˆ์Šต๋‹ˆ๋‹ค.
  • Reset์€ destructive role ๋ฒ„ํŠผ๊ณผ ํ™•์ธ alert๋ฅผ ๊ฑฐ์ณ DrinkWaterViewModel.reset()์„ ํ˜ธ์ถœํ•ฉ๋‹ˆ๋‹ค.
  • ์‹ค์ œ ์‚ญ์ œ ์ •์ฑ…์€ ๊ธฐ์กด HealthKit ์›๋ณธ/์•ฑ ์ƒ์„ฑ ๊ธฐ๋ก ์‚ญ์ œ ํ๋ฆ„์„ ์œ ์ง€ํ•˜๊ณ , ์ด๋ฒˆ ๋ณ€๊ฒฝ์€ ์˜ค์กฐ์ž‘ ๋ฐฉ์ง€์™€ ์•ˆ๋‚ด ๊ฐ•ํ™”์— ์ง‘์ค‘ํ–ˆ์Šต๋‹ˆ๋‹ค.

๐Ÿ“ธ Screenshots / Videos

Before

์ฒจ๋ถ€ ์—†์Œ

After

์ฒจ๋ถ€ ์—†์Œ

โœ… Testing

Test Plan

  • ์•ฑ ๋นŒ๋“œ ์„ฑ๊ณต
  • Debug ๋ชจ๋“œ ์ •์ƒ ์ž‘๋™
  • Release ๋ชจ๋“œ ์ •์ƒ ์ž‘๋™
  • ๊ธฐ์กด ๊ธฐ๋Šฅ ์˜ํ–ฅ ์—†์Œ
  • Widget Extension ์ •์ƒ ์ž‘๋™ (ํ•ด๋‹น์‹œ)

Test Environment

  • iOS Version: iOS Simulator 26.5
  • Device: iPhone 17 Pro Simulator (125B9A31-6A4A-466D-A8AB-410730B22446)
  • Xcode Version: Xcode 26.5 (17F42)

Test Results

  • git diff --check ํ†ต๊ณผ
  • jq empty Project/Shared/Localization/Resources/Localizable.xcstrings ํ†ต๊ณผ
  • make lint ํ†ต๊ณผ
  • make arch-check ํ†ต๊ณผ
  • xcodebuild test -workspace Mulimi.xcworkspace -scheme PresentationLayer -destination 'platform=iOS Simulator,id=125B9A31-6A4A-466D-A8AB-410730B22446' -sdk iphonesimulator -derivedDataPath /tmp/MulimiIssue234PresentationTests ํ†ต๊ณผ (88 tests)
  • xcodebuild build -workspace Mulimi.xcworkspace -scheme Mulimi -destination 'generic/platform=iOS' CODE_SIGNING_ALLOWED=NO CODE_SIGNING_REQUIRED=NO -derivedDataPath /tmp/MulimiIssue234AppBuild ํ†ต๊ณผ
  • ์ฐธ๊ณ : ์ตœ์ดˆ iPhone 16 Pro, OS=latest ํ…Œ์ŠคํŠธ ์‹คํ–‰์€ ํ•ด๋‹น ์‹œ๋ฎฌ๋ ˆ์ดํ„ฐ๊ฐ€ ์—†์–ด ์‹คํŒจํ–ˆ๊ณ , ์‚ฌ์šฉ ๊ฐ€๋Šฅํ•œ iPhone 17 Pro ์‹œ๋ฎฌ๋ ˆ์ดํ„ฐ๋กœ ์žฌ์‹คํ–‰ํ•ด ํ†ต๊ณผํ–ˆ์Šต๋‹ˆ๋‹ค.

๐Ÿšจ Breaking Changes

  • Yes (์•„๋ž˜์— ์„ค๋ช…)
  • No

๐Ÿ“Œ Additional Notes

  • HealthKit ์‚ญ์ œ ๋ฒ”์œ„ ์ž์ฒด๋Š” ๊ธฐ์กด ์•ฑ ์ƒ์„ฑ ๊ธฐ๋ก ์ •์ฑ…์„ ์œ ์ง€ํ•ฉ๋‹ˆ๋‹ค.
  • ์•ฑ ๋นŒ๋“œ ์ค‘ AppIntents SSU artifact archive ๋กœ๊ทธ๊ฐ€ ์ถœ๋ ฅ๋์ง€๋งŒ ์ตœ์ข… ๊ฒฐ๊ณผ๋Š” BUILD SUCCEEDED์˜€์Šต๋‹ˆ๋‹ค.

๐Ÿ‘€ Review Checklist

  • ์ฝ”๋“œ๊ฐ€ ํ”„๋กœ์ ํŠธ์˜ ์ฝ”๋”ฉ ์ปจ๋ฒค์…˜์„ ๋”ฐ๋ฅด๊ณ  ์žˆ๋‚˜์š”?
  • ์ƒˆ๋กœ์šด ์˜์กด์„ฑ์ด ํ•„์š”ํ•œ๊ฐ€์š”? ์•„๋‹ˆ์š”
  • ๋ฌธ์„œ ์—…๋ฐ์ดํŠธ๊ฐ€ ํ•„์š”ํ•œ๊ฐ€์š”? ๋ฐ˜์˜ ์™„๋ฃŒ
  • ๋ฐ์ดํ„ฐ๋ฒ ์ด์Šค ๋งˆ์ด๊ทธ๋ ˆ์ด์…˜์ด ํ•„์š”ํ•œ๊ฐ€์š”? ์•„๋‹ˆ์š”
  • ํ™˜๊ฒฝ ์„ค์ • ๋ณ€๊ฒฝ์ด ํ•„์š”ํ•œ๊ฐ€์š”? ์•„๋‹ˆ์š”

์˜ค๋Š˜ ๊ธฐ๋ก ์ดˆ๊ธฐํ™” ํ™•์ธ UX ๊ฐœ์„ 
@gaeng2y gaeng2y merged commit 6a9fcaf into develop May 28, 2026
3 checks passed
@gaeng2y gaeng2y deleted the feature/#234-reset-ux branch May 28, 2026 12:32
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