Skip to content

[mod/#59] 자체 qa 2차 반영#60

Merged
sonms merged 3 commits intodevelopfrom
feat/#59-qa-reflect-three
Jan 20, 2026
Merged

[mod/#59] 자체 qa 2차 반영#60
sonms merged 3 commits intodevelopfrom
feat/#59-qa-reflect-three

Conversation

@sonms
Copy link
Member

@sonms sonms commented Jan 20, 2026

ISSUE

❗ WORK DESCRIPTION

  • 자녀 id 저장 후 불러오기 기능 추가 - UserInfoManager - getChildId
  • 부모 로그아웃 기능 붙여넣기
  • escape 처리

📢 TO REVIEWERS

  • 홧팅홧팅

📸 SCREENSHOT

sonms added 2 commits January 20, 2026 23:27
- `KieroNavHost`에 `navigateToSelection`을 추가하여 자녀 선택 화면으로 이동하는 로직을 구현했습니다.
- 부모 로그인 시 첫 번째 자녀의 ID를 `UserInfoManager`를 통해 로컬에 저장하도록 수정했습니다.
- `DataStoreConstant`에 `KEY_CHILD_ID`를 추가했습니다.
- 하단 네비게이션 바(`MainBottomBar`)의 가시성 로직을 `isParentMode`로 명확히 하고, 스낵바의 하단 패딩을 조정했습니다.
- `UserInfoManager`에 `childId`를 저장하고 조회하는 기능을 추가했습니다.
- 스플래시 화면에서 자녀 정보를 가져온 후, 첫 번째 자녀의 ID를 `UserInfoManager`에 저장하도록 로직을 수정했습니다.
- 일정 및 미션 관련 ViewModel에서 `childId`를 `UserInfoManager`에서 직접 가져와 사용하도록 변경했습니다.
- 부모 홈 화면과 회원가입 완료 화면에 로그아웃 기능을 추가하고, 로그아웃 시 토큰과 데모 데이터를 삭제하는 로직을 구현했습니다.
- 미션이 없을 경우 `Empty` 상태를 표시하도록 `ParentMissionViewModel`을 수정하고, 관련 UI를 `ParentMissionScreen`에 추가했습니다.
@sonms sonms requested a review from a team January 20, 2026 14:30
@sonms sonms self-assigned this Jan 20, 2026
@sonms sonms requested review from dmp100, seungjae708 and vvan2 and removed request for a team January 20, 2026 14:30
Copy link
Member

@vvan2 vvan2 left a comment

Choose a reason for hiding this comment

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

Datastore 에 넣어주신거 감사합니다...

- `ParentMissionScreen`과 `ParentScheduleScreen`에서 `LaunchedEffect`를 사용하여 데이터 로딩 시점을 초기화 시점으로 변경했습니다.
- `ParentMissionViewModel`과 `ParentScheduleViewModel`에서 `init` 블록 내의 데이터 호출 로직을 제거했습니다.
- 자동 미션 추가(`ParentAutoAddScreen`) 시 `childId`를 파라미터로 받지 않고, `UserInfoManager`에서 직접 조회하여 사용하도록 수정했습니다.
- `analyzeNotice` 호출 시, `noticeText`의 개행문자와 따옴표를 이스케이프 처리하도록 수정했습니다.
- `ParentAutoAddScreen`의 `SideEffect` 처리 로직을 `collectSideEffect` 확장 함수와 `LocalGlobalUiEventTrigger`를 사용하도록 변경하여 코드 스타일을 통일했습니다.
Copy link
Collaborator

@dmp100 dmp100 left a comment

Choose a reason for hiding this comment

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

이스케이프 처리 감사합니다.. 감사합니다..

Copy link
Collaborator

@seungjae708 seungjae708 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 나중에 천천히 다시 볼게용

@sonms sonms merged commit c38ace3 into develop Jan 20, 2026
@vvan2 vvan2 added this to the [키어로] AppJam milestone Jan 21, 2026
@vvan2 vvan2 changed the title [mod] 자체 qa 2차 반영 [mod/#59] 자체 qa 2차 반영 Jan 21, 2026
@vvan2 vvan2 added mod 🎉 큰 단위의 코드 수정 🪐민성 민성 전용 라벨 labels Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod 🎉 큰 단위의 코드 수정 🪐민성 민성 전용 라벨

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[mod] 자체 QA 2

4 participants