Skip to content

Commit

Permalink
Merge pull request #259 from GEON-PPANG/fix/#255-memory-issue-at-onbo…
Browse files Browse the repository at this point in the history
…arding

[Fix] #255 - 온보딩 쪽에 있던 메모리 이슈 해결
  • Loading branch information
seongmin221 authored May 16, 2024
2 parents 47665de + 1e31d60 commit 0c97928
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 107 deletions.

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -278,7 +278,6 @@ extension LoginRequiredViewController: ASAuthorizationControllerDelegate {
nickname: ""
)

guard let presenting = self.presentingViewController as? UINavigationController else { return }
self.postSignUp(with: request, viewController: self) { role in
KeychainService.setKeychain(of: .role, with: role)
self.dismiss(animated: true) {
Expand All @@ -288,6 +287,7 @@ extension LoginRequiredViewController: ASAuthorizationControllerDelegate {
}
} else {
DispatchQueue.main.async {
guard let presenting = self.presentingViewController as? UINavigationController else { return }
Utils.push(presenting, NickNameViewController())
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -306,9 +306,9 @@ extension OnboardingViewController: ASAuthorizationControllerDelegate {
nickname: ""
)

self.postSignUp(with: request) { role in
self.postSignUp(with: request) { [weak self] role in
KeychainService.setKeychain(of: .role, with: role)
self.check(role: role)
self?.check(role: role)
}

case .revoked:
Expand Down

0 comments on commit 0c97928

Please sign in to comment.