Skip to content

[mod/#82] last appjam qa#89

Merged
sonms merged 11 commits intodevelopfrom
mod/#82-last-appjam-qa
Feb 18, 2026
Merged

[mod/#82] last appjam qa#89
sonms merged 11 commits intodevelopfrom
mod/#82-last-appjam-qa

Conversation

@sonms
Copy link
Member

@sonms sonms commented Feb 13, 2026

ISSUE

❗ WORK DESCRIPTION

  • 백스택에 스플래시가 남아있는 현상 수정
  • 부모 로그인 후 스케줄에서 뒤로가기 시 다시 초대코드 뷰로 돌아가는 스택 꼬임 현상 수정
  • extension으로 만든 dropshadow 를 android의 dropshadow로 구현
  • 유저 로그아웃 시 저장 내용 제거 및 sse 연결 종료

📢 TO REVIEWERS

  • 씩씩하게 수정했습니다 (규현아 todo 확인해줭)
  • 주완형도 toto 확인해줭 @vvan2

- 부모 로그인 화면(`AuthParentScreen`)에서 Back-press 이벤트 발생 시, 이전 화면인 역할 선택 화면(`SelectionScreen`)으로 이동하도록 수정했습니다.
- 뒤로가기 시 `SseManager`의 구독을 중지하는 로직을 추가했습니다.
- `BackHandler`의 `onBack` 콜백을 `viewModel::navigateUp`으로 직접 호출하도록 변경했습니다.
- `ParentAlarmCard` 컴포저블에서 `AsyncImage`를 공용 `UrlImage` 컴포넌트로 교체했습니다.
- `imageUrl`의 타입을 `Any?`에서 `String?`으로 변경하여 타입 안전성을 강화했습니다.
- 로그아웃 시 토큰뿐만 아니라 부모 정보(`ParentInfo`)도 함께 삭제하도록 로직을 개선했습니다.
@sonms sonms requested a review from a team February 13, 2026 07:59
@sonms sonms self-assigned this Feb 13, 2026
@sonms sonms requested review from dmp100, seungjae708 and vvan2 and removed request for a team February 13, 2026 07:59
@sonms sonms added mod 🎉 큰 단위의 코드 수정 🪐민성 민성 전용 라벨 labels Feb 13, 2026
@vvan2 vvan2 added this to the [키어로] 1차 스프린트 milestone Feb 13, 2026
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.

백스택 처리와 SSE 관리 등 까다로운 부분들 잘 처리해주셨네요 고생하셨습니당~

import com.kiero.R

@Composable
fun UrlImage(
Copy link
Collaborator

Choose a reason for hiding this comment

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

p1) 함수명이 파일명이랑 달라요!

Copy link
Member Author

Choose a reason for hiding this comment

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

허걱 감사합니당

Copy link
Member Author

Choose a reason for hiding this comment

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

qa의 승재 역시 믿고봅니다

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.

오류 몇가지 있던거 고쳐주셨군요.. 대 민 성
어푸푸

@dmp100
Copy link
Collaborator

dmp100 commented Feb 15, 2026

Todo 확인했어용 ! 풀 받아서 계속 진행해보겠습니당

@vvan2 vvan2 changed the title [Mod/#82] last appjam qa [mod/#82] last appjam qa Feb 18, 2026
@sonms sonms merged commit c458929 into develop Feb 18, 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

4 participants