Skip to content
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

[#83] 스크린샷 잘못 올렸을 때 alert들 추가하기 #86

Merged
merged 7 commits into from
Nov 12, 2024

Conversation

Minkyeong-Choi
Copy link
Collaborator

@Minkyeong-Choi Minkyeong-Choi commented Nov 12, 2024

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요

  • 홈뷰에서 사진을 계속 잘못올려도 계속 알러트가 뜰 수 있게 수정하였습니다.
  • ScannedJourneyInfo뷰에서 Reupload할때 잘못된 사진을 올리면 경고 메세지가 뜰 수 있게 변경하였습니다.

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 첫번째 올바른 사진을 올리고, 두번째 잘못된 사진을 올리고, 세번째 잘못된 사진을 올렸을 때, 두번째에는 올바르게 경고창이 떴다가 세번째에 잘못된 사진을 올렸을 때는 경고창이 뜨지 않는 에러가 있습니다...
  • 방금 또 발견했는데, 하나라도 글씨가 있으면 두번째 잘못된 사진을 올렸을 때 경고조차 뜨지 않는 에러를 발견했습니다..?! -> 이거는 제가 빠르게 해결해볼게요

@Minkyeong-Choi Minkyeong-Choi self-assigned this Nov 12, 2024
@Minkyeong-Choi Minkyeong-Choi linked an issue Nov 12, 2024 that may be closed by this pull request
@Minkyeong-Choi Minkyeong-Choi linked an issue Nov 12, 2024 that may be closed by this pull request
@Minkyeong-Choi
Copy link
Collaborator Author

모두 해결되었습니다,,

Comment on lines +16 to 17
@State private var newScannedInfo: String = ""
@Binding var scannedJourneyInfo: String
Copy link
Collaborator

Choose a reason for hiding this comment

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

두 변수 다 사진에서 스캔해 온 정보를 담고 있는 변수로 보이는데, 하나의 변수로 관리하는 건 힘든가요 ~?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

scannedJourneyInfo 변수가 HomeView에서 정의되는데, 이 변수를 이 뷰에서 그대로 사용하면 문자열이 빌 때 알아서 HomeView 분기처리에 따라 업데이트 되더라고요... 그래서 ScannedJourneyInfoView 안에서만 사용할 변수를 따로 만들었습니다.

Copy link
Collaborator

Choose a reason for hiding this comment

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

�흠 그렇군요 ....
분기 처리 부분을 먼저 리팩하고나서 변수 다시 살펴보도록 합시댜 ~

Copy link
Collaborator

@dbqls200 dbqls200 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 !

@Minkyeong-Choi Minkyeong-Choi merged commit 7f9e043 into dev Nov 12, 2024
@Minkyeong-Choi Minkyeong-Choi deleted the fix/83-wrong-screenshot-alert branch November 14, 2024 17:05
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.

ScannedJourneyInfoView 잘못된 사진을 올렸을 때 버그 HomeView Alert 관련 버그
2 participants