-
Notifications
You must be signed in to change notification settings - Fork 0
[RELEASE] v1.1.2 To Main merge #167
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
Conversation
- 모든 env 파일 무시 규칙 추가
- DIContainer 의존성 추가 - 프로젝트에 필요한 의존성 추가 - Factory를 이용한 화면 전환으로 변경
- 화면 전환에 필요한 화면을 별도의 Mock 객체로 대체
- 임시로 지정해준거고 기존 코드들에 문제없게 해두었습니다. - 추후 디자인 시스템 개편 과정에서 참고하도록 하겠습니다.
- 로그인 화면과 서브 로그인 화면은 일부 컴포넌트의 속성을 제외하고는 동일한 동작을 가져감 - 서브로그인과 로그인을 리액터의 상태에 기반하여 구분되도록 수정 - 추후 로그인 개선시 하나의 코드만 수정해도 동일하게 동작하도록 개선
- 요구사항 분석과정에서 코멘트의 변경이 main/sub 만으로 나뉘지 않을것으로 예측함 - 기존에 isSubLogin이라는 bool로 분기를 하던것도 LoginSceneType을 별도로 선언하여 타입 추가에 대한 대비
…recent-login-duplication # Conflicts: # Poppool/PresentationLayer/SearchFeature/SearchFeature.xcodeproj/project.pbxproj
- 로그아웃이나 회원탈퇴시 정보 제거가 이루어지지 않고 있었음 - 해당 로직을 추가함
- View에서 Reactor의 인스턴스(userDefautService)를 직접 접근하는것을 방지 - Rx의 ViewController LifeCycle을 이용하도록 수정
…m/PopPool/iOS into fix/#160-recent-login-duplication
[FIX] 로그인 툴팁이 중복을 발생하던 문제 수정
…/v1.1.2 # Conflicts: # Poppool/Poppool.xcworkspace/xcshareddata/swiftpm/Package.resolved
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
zzangzzangguy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
릴리즈버전 병합을 위한 PR