[Refactor] 토큰 재발급 로직의 오류를 수정하였고 일부 디자인을 변경했습니다.#102
Merged
MinwooJe merged 6 commits intoGDSC-Popcorn:developfrom Apr 10, 2025
Merged
[Refactor] 토큰 재발급 로직의 오류를 수정하였고 일부 디자인을 변경했습니다.#102MinwooJe merged 6 commits intoGDSC-Popcorn:developfrom
MinwooJe merged 6 commits intoGDSC-Popcorn:developfrom
Conversation
- 뷰파일, 뷰컨트롤러파일에서 코드 삭제
- 키체인에 저장하는 로직 변경 - 키체인에서 delete -> add 하는 로직을 이미 존재하면 update하는 로직으로 변경
MinwooJe
requested changes
Apr 9, 2025
Member
MinwooJe
left a comment
There was a problem hiding this comment.
p1)
회원가입 화면2에서 뒤로가기 버튼을 눌렀는데도 여전히 데이터가 존재한다는 건 회원가입 화면 2 뷰컨트롤러가 메모리에서 해제되지 않는 것 같애요...
내비게이션 컨트롤러에서 popViewController를 호출하면 해당 뷰 컨트롤러는 메모리에서 해제되는데... 디버깅이 필요한 것 같습니다ㅠ
회원가입 2로 넘어가려고 하니 이메일 인증이 지금 안되는 것 같애서 확인할 수 가 없네요ㅠ
- 회원가입화면2에서 resetData를 하는 메서드 대신 DIContainer에서 뷰컨트롤러를 관여하지 않도록 로직 변경
MinwooJe
approved these changes
Apr 10, 2025
Member
MinwooJe
left a comment
There was a problem hiding this comment.
DIContainer에서 뷰컨트롤러를 생성해서 메모리 해제가 안됐던거네요! 수고하셨습니다~
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
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.
배경
작업 내용
- 회원가입화면2의 상태 초기화를 위해resetSignUpData()메서드를 추가하였습니다.테스트 방법
임시로 사용하였던 다음 코드도 변경되었습니다.
리뷰 노트