Skip to content

Conversation

@cirtuare
Copy link
Contributor

@cirtuare cirtuare commented Jul 30, 2025

🐿️ Pull Requests

🪵 작업 브랜치

🥔 작업 내용

3번째 인증 지역 추가 시 크래시나는 오류 해결했습니다.

🚨 참고 사항

허허.. 조금 더 신경써서 코드 짤게요 .. ㅠ

📸 스크린샷

image

💥 To be sure

  • 모든 뷰가 잘 실행되는지 다시 한 번 체크해주세요 !

🌰 Resolve issue

@cirtuare cirtuare self-assigned this Jul 30, 2025
@cirtuare cirtuare added the 🆘 hotfix hot fix label Jul 30, 2025
@cirtuare cirtuare requested a review from yurim830 July 30, 2025 06:34
@cirtuare cirtuare added the 🍓 수민 수민 label Jul 30, 2025
@cirtuare cirtuare changed the title [HOTFIX] 3번째 인증 지역 추가 시 크래시나는 오류 해결 (#236) [HOTFIX] 3번째 인증 지역 추가 시 크래시나는 오류 해결 (#225) Jul 30, 2025
@cirtuare cirtuare changed the title [HOTFIX] 3번째 인증 지역 추가 시 크래시나는 오류 해결 (#225) [HOTFIX] 3번째 인증 지역 추가 시 크래시나는 오류 해결 (#236) Jul 30, 2025
Copy link
Collaborator

@yurim830 yurim830 left a comment

Choose a reason for hiding this comment

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

어쩌다가 발견했대~~~ 출시 전에 잡아서 다행이네요!! 고생하셨씁니다!!

Comment on lines 122 to 123
print(viewModel.verifiedAreaList)
print(indexPath.item)
Copy link
Collaborator

Choose a reason for hiding this comment

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

🐿️ 혹시 프린트문은 일부러 남겨두신걸까요?! 크게 의미 없으면 삭제하고 머지해도 좋을 것 같아요~!

} else {
if !viewModel.verifiedAreaList.isEmpty {
cell.bindData(viewModel.verifiedAreaList[indexPath.item].name, indexPath.item)
if indexPath.item < viewModel.verifiedAreaList.count {
Copy link
Collaborator

Choose a reason for hiding this comment

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

🐿️🐿️ 질문이요! 이 조건이 충족되지 않는 경우도 있나요?
항상 indexPath.itemdms verifiedArea 개수보다 작을 것 같은데 조건을 거신 이유가 궁금합니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

"지역 추가하기" 버튼도 컬럭션 뷰 아이템이라, 지역추가하기 버튼이 있다면 그 때 indexPath.items = verifiedAreasList.count이에요!

@cirtuare cirtuare merged commit ecd8aeb into develop Jul 30, 2025
@cirtuare cirtuare added this to the Sprint - 4 milestone Aug 3, 2025
@cirtuare cirtuare deleted the hotfix/#236 branch August 12, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[HOTFIX] 3번째 지역인증 추가 시 추가 안 되는 에러 해결

3 participants