Skip to content

[FIX] CardShredView 네비게이션 화면전환 적용 및 MainTabView 개선 - #44

Merged
cchanmi merged 12 commits into
developfrom
fix/trashList
Sep 2, 2026
Merged

cchanmi merged 12 commits into
developfrom
fix/trashList

Conversation

@dlrjswns

Copy link
Copy Markdown
Contributor

Describe

CardShredView를 네비게이션 화면전환 적용했고 MainTabView에서 루트 NavigationStack을 관리할 수 있도록 수정했습니다.

Works made

  1. CardShredView 표출 시 네비게이션 화면 전환 적용
  2. MainTabView에서 NavigationStack 관리 적용 및 중복 코드 삭제
  3. CardDetailView 하단에서 위로 움직이는 애니메이션에서 중앙 표출 애니메이션 적용

Changes Made

As-Is

기존 로직

  • ChatView에서 띄워진 CardDetailView 하단에서 위로 띄워지듯이 표출
  • 선택한 카드 혹은 전체 카드 파쇄 시 이동하는 CardShredView 모달 표출
  • 뷰마다 toolbar를 통해 네비바 및 탭바 히든 처리

To-BE

동작 영상
https://github.com/user-attachments/assets/305641a5-c07f-45bb-b8c9-e55091d81a5b

How to Test

  1. 생성된 카드 클릭 후 띄워진 CardDetailView 확인
  2. 기록버리기 버튼 클릭 후 파쇄 이후 ArchiveDetailView로 이동 확인
  3. MainTabView적용된 RootView들외 탭바 hidden처리 확인
  4. 프로젝트 내 기본 네비바 사라졌는지 확인

Issues Resolved

Additional context

References

@dlrjswns
dlrjswns requested a review from cchanmi August 24, 2026 14:25
@dlrjswns dlrjswns self-assigned this Aug 24, 2026
@dlrjswns dlrjswns added the enhancement New feature or request label Aug 24, 2026
Comment on lines +77 to +80
try await DefaultAuthRepository(
networkManager: NetworkManager.shared,
tokenStorage: TokenStorage.shared
).autoLogin()

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

p1

지금 현재 구조가 Core -> Data 구현제를 아는 구조인 것 같아요

NetworkManager가 DefaultAuthRepository를 직접 생성하고 있는데, 이미 기존 코드에서 DefaultAuthRepository가 NetworkManger로 요청을 보내는 반대 방향 의존도 있어서 결과적으로 Core -> Data -> Core 로 순환이 생기고 있는 것 같아요

NetworkManager에서 AuthRepository 프로토콜을 의존하고 구현체는 외부에서 주입받는 형태로 바꾸어도 좋을 것 같습니다


// 재발급된 토큰은 HttpHeader가 요청을 다시 만들 때 Keychain에서 새로 읽어오므로,
// 원래 요청을 그대로 한 번 더 시도하면 된다.
return try await request(endpoint, responseType: responseType, isRetryAfterReissue: true)

@cchanmi cchanmi Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

p1

reissue 실패 → autoLogin 폴백 로직이 핵심 수정사항인 것 같은데 관련 테스트 코드가 있으면 좋을 것 같아요

성공/실패(missingFirebaseUser, firebaseSignInFailed) 케이스만이라도 커버되면 좋을 것 같고... 아니면 따로 검증할 방법 있을까요?

@cchanmi

cchanmi commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

추가로 확인 필요한 부분이... 검색 화면에서 입력 텍스트필드에 텍스트 입력할 때 폰트 설정 필요해 보이네요
지금은 기본 폰트여서 글씨가 피그마보다 크게 보이는 것 같아요

@cchanmi
cchanmi merged commit 0986c7d into develop Sep 2, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants