-
Notifications
You must be signed in to change notification settings - Fork 0
Feat/#55 - CameraView UI + 기능 모두 구현 #77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
what? - CameraView 기능을 구현 시 필요한 리소스들을 추가했습니다.
what? - CameraScene 뷰를 연결 하기 위해 PhotoDetailsScene 과 ScanLoadScene 폴더 및 내부 더미 파일을 추가했고, AppROute, RootView,ModuleFactory 를 변경했습니다.
what? - 썸네일 화면에 촬영 장수를 표시할 뱃지를 구현했습니다. modifier 로 쓸 수 있게 구현했습니다.
what? - CameraFeature 을 구현하면서, 중간 액션과 리턴 액션이 없는 부분이 너무 불편했습니다. (미니가 추가해주셨다고 해서 추후 반영 예정입니다.) Model은 Util/Camera 에서 캡쳐된 모델을 가져오기 위해 추가해두었습니다.
what? - MapView 에서 CameraView 로 전환할 때 caseID를 가져오도록 하기 위해 코드를 일부 수정했습니다. 테스트 할 때 caseID를 못불러와서 임시로 UUID를 넣어 구현했습니다.
what? - CameraView 에서 사용할 컴포넌트들을 추가했습니다.
what? - Util/Camera 에서 func로 모조리 붙어있던 기능들을 extension 으로 분리했습니다. - CameraView 에서 사용하기 위한 CameraServiceService 도 일부 로직 문제가 있어 수정했습니다.
what?
- CameraView 를 구현했습니다. 여기서 이슈가 발생.. confirmDialog 에 저렇게 작성하지 않으면 취소 버튼이 안나오는 이슈가 있었습니다.
.confirmationDialog(
"추출된 주소가 없습니다",
isPresented: $showRetryDialog,
titleVisibility: .visible
) {
Button("다시 촬영") {
handleRetry()
}
Button("취소", role: .cancel) {
handleCancel()
}
} message: {
Text("다시 촬영하시겠습니까?")
}
이렇게 role 에 cancel만 줘도 안뜨는 기적..
해결방법을...찾고싶다....
what? - PhotoDetails 뷰를 연결 하기 위해 CameraView 일부와 AppROute, RootView,ModuleFactory 를 변경했습니다.
what? - PhotoDetails 뷰 구현을 위해 PhotoDetailsHeader, ZoomableImageView, ZoomState 를 추가했습니다
what? - PhotoDetails 뷰 구현을 위해 View, Feature 을 구현했는데 CameraVIew 와 photos를 동기화 시키기 위해 cameraModelf을 데리고 다니고 있어서 이를 해결해야할 것 같습니다.
what? system 폰트로 들어있어서 수정했습니다.
This reverts commit 010ad7c.
This reverts commit 1f8fcbc.
This reverts commit 52cf3be.
YooGyeongMo
approved these changes
Nov 10, 2025
Collaborator
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p1. 요청 주신 부분 코드 찾아보니까 전체에서 안떠서 제가 TCA적으로 생각해보면
Effect 취소 신호를 안받고 store에서 해당 effect를 취소해도, Task는 자기 할일 끝날때 까지 계속돌게 하는거랑
에러랑 완료처리 경계를 좀 더 명확하게 할 필요가 있어보여요.
return .run { [camera] send in
guard let stream = await camera.getDocumentDetectionStream() else {
// 필요하다면 에러/종료 액션 보낼 수 있음
return
}이런식으로 바꿔보는거 어떤가요 ?
MuchanKim
reviewed
Nov 10, 2025
SUSA24-iOS/SUSA24-iOS/Sources/Util/Vision/Extensions/KoreanAddressPattern.swift
Show resolved
Hide resolved
SUSA24-iOS/SUSA24-iOS/Sources/Presentation/CameraScene/CameraFeature.swift
Show resolved
Hide resolved
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Summary
CameraScene 기능 구현
🔨 What
문서를 촬영하는 뷰를 구현했습니다.

👀 Review Notes
confirmDialog 에 저렇게 작성하지 않으면 취소 버튼이 안나오는 이슈가 있었습니다.
이렇게 role 에 cancel만 주면 자꾸만 안뜨는 기적.. 해결방법을...찾고싶다....
ㅎ.. 이부분 살려주세요. (안그래도 미니가 merge 기능 넣어줬대서 나중에 pull 받아서 처리해야할 듯)
아니.. 프레임 계속 띄우고 해야하는데 중간 액션이 안되어서 너무 머리가 터질 것 같았어요.. 액션을 추가하자니 갈길이 멀고
어제 고민하다가 그냥 냅다 for문 돌려버림... 곧 바꿀게요 🤨