Skip to content

Commit 18c10b5

Browse files
authored
[NL-89] 약관 선택 관련 버그 수정 (#66)
* [NL-89]: 약관 전체 선택 후, 개별 해제 시 전체 선택 상태가 유지되는 버그 * [NL-89]: 사용자 생성 시 임시 로딩 처리 추가
1 parent 5961c45 commit 18c10b5

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

Feature/Onboarding/Sources/Agreement/AgreementViewController.swift

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ final class AgreementViewController: BaseViewController {
2929

3030
private var isAllAgree: Bool {
3131
return agreementItems.filter { $0.isRequired }.allSatisfy { $0.isAgreed }
32-
|| agreementItems[0].isAgreed
3332
}
3433

3534
private lazy var containerView = UIView().then {
@@ -152,16 +151,17 @@ extension AgreementViewController {
152151
confirmButton.addTarget(self, action: #selector(confirmButtonTapped), for: .touchUpInside)
153152
}
154153

155-
private func updateConfrimButton() {
154+
private func updateConfirmButton() {
156155
confirmButton.backgroundColor = isAllAgree ? STColors.primary2.color : STColors.primary7.color
157-
if isAllAgree { confirmButton.isEnabled = true } else { confirmButton.isEnabled = false }
156+
confirmButton.isEnabled = isAllAgree
158157
}
159158

160159
@objc private func closeButtonTapped() {
161160
self.dismiss(animated: false, completion: nil)
162161
}
163162

164163
@objc private func confirmButtonTapped() {
164+
showLoading() // TODO: 나중에 정리 필요
165165
delegate?.agreementViewDidComplete()
166166
}
167167

@@ -231,11 +231,12 @@ extension AgreementViewController: AgreementCellDelegate {
231231
for i in 0..<agreementItems.count {
232232
agreementItems[i].isAgreed = shouldAgreeAll
233233
}
234-
collectionView.reloadData()
235234
} else {
236235
agreementItems[indexPath.item].isAgreed = isAgreed
236+
agreementItems[0].isAgreed = isAllAgree
237237
}
238-
updateConfrimButton()
238+
collectionView.reloadData()
239+
updateConfirmButton()
239240
}
240241
}
241242
}

0 commit comments

Comments
 (0)