Skip to content

Feature/login: 화면 UI 추가 및 AppFlowCoordinator 화면 진입 부분과 BadgeIconType 에 return 추가#29

Open
cestbonciel wants to merge 29 commits intodevelopfrom
Feature/login
Open

Feature/login: 화면 UI 추가 및 AppFlowCoordinator 화면 진입 부분과 BadgeIconType 에 return 추가#29
cestbonciel wants to merge 29 commits intodevelopfrom
Feature/login

Conversation

@cestbonciel
Copy link
Copy Markdown
Member

@cestbonciel cestbonciel commented Mar 22, 2024

Motivation ⍰

  • 다들 바쁘실텐데 일교차 조심하시고요. 시간되실 때 코드 봐주시면 감사하겠습니다. 주말 잘보내세요.

Key Changes 🔑

  1. 로그인 화면이 나오나 홈뷰로 이동가능하게끔 우측상단에 임시 홈 이동 파란색버튼을 달았습니다.
    차후 로그인 로직 작업 후 appFlowCoordinator 는 기본 원래 로직대로 되게끔 바꾸려고 합니다.
    AppFlowCoordinator.swift
extension AppFlowCoordinator: CoordinatorFinishDelegate {
    func coordinatorDidFinish(childCoordinator: Coordinator) {
        childCoordinators = childCoordinators.filter({ $0.type != childCoordinator.type })
        
        switch childCoordinator.type {
        case .tab:
            navigationController.viewControllers.removeAll()

            // TODO: 토큰 삭제
            Keychain.shared.delete(.accessToken)

            showLogin()
        case .login:
            navigationController.viewControllers.removeAll()

            // TODO: 토큰 임시 설정
            Keychain.shared.set("test", forKey: .accessToken)

            showTabBar()
        default:
            break
        }
    }
}
  1. 최신화된 develop 에 충돌 수정하고 작업하려고 했는데, 혹시 모르니 작업하셨던 부분이랑 충돌되는게 있는지만 봐주시면 될 거같아요.

    3.⭐️ 제가 제가 건드려야 할 코드 외에 수정한 부분은 AppFlowCoordinator 랑 릴리가 작업하신 BadgeIconTypereturn 을 달아둔 부분 두 군데 입니다. return 은 달아도 괜찮다고 다들 말씀하셔서 달고 commit 해요.

To Reviewers 🙏🏻

  1. 심각하게 이건 통일성이 없다 하는 부분은 짚어주시면 수정할게요. BaseView 나 BaseVC 로 작업하지 않아서 코드가 통일 되지 못했어요 이것도 차후 작업하려고 합니다!
  2. 큰 오류나 잘못된 부분이 있다면 화면 보시고 말씀해주세요 마지막 지역선택 버튼은 액션이 아직 들어가면 안되는데 무한으로 지역선택되니 그건 지역선택 UI 하면서 꼭 수정할게요.

1. 기본 UI 만 작업하였고, 다음 버튼을 누르는 것만 달아놨습니다. 경험, 레벨, 지역, 장르 선택할때 (setSelecte)활성화 버튼 선택작업은 아직 안되었어요.
2. 차후 작업할 UI는 지역선택 화면으로 작업할 예정입니다.
2. 애플 로그인 부분은 제대로 화면 이동은 안되고 예전에 달아 놨던 코드라서 나중에 수정할 예정입니다.
capability - sign in with apple 만 달아놓고, keychain 외 몇 개 파일 분리해서 추가했습니다.

* 그 외 git rebase 로 해보려고 했는데 merge 에 익숙해서 이미 merge 하고, 충돌 부분 수정하고, git push 하고 PR 날리네요.

Linked Issue 🔗

Chore: 지워진 Login codes 되돌리기
This reverts commit dc22d4d.
Chore: 되돌리기 ( merge from github on 3, mar)
This reverts commit cf27b7e.
Fix: 지워진 파일 다시 병합
Chore: merge from develop with myPage
Chore: tuist 재설치 후 merge from github
This reverts commit 7882d95.

Fix: HomeViewController 잘못된 코드 이전으로 돌리기
@cestbonciel cestbonciel requested review from Minny27 and hililyy March 22, 2024 11:25
@cestbonciel cestbonciel requested a review from Jeon0976 March 22, 2024 11:25
@cestbonciel cestbonciel self-assigned this Mar 22, 2024
let moveView: () -> Void
}

protocol SignUpViewModelMove {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

이 부분은 어떤걸 의미하시는지 알수있을까요??

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

sns 로그인에서 회원 추가 정보값 입력(경험,레벨,관심지역,장르,프로필입력까지) 이 단계적으로 있어서 화면 이동시 쓸까 해서 만들었던 것 같습니다.

@Jeon0976
Copy link
Copy Markdown
Contributor

Jeon0976 commented Mar 24, 2024

  • SignUpView내부에서 통일성을 주기위해 파일명을 변경했습니다. 앞에 SignUp + 내용 + Selection 으로 변경했습니다. 확인 한 번 부탁드릴께요.

  • 기존 ChoiceRegionOfInterestViewController에서 SignUpRegionSelectionViewController 으로 변경한 VC에서 지역선택 action을 진행하면 viewModel이 없어서 에러처리가 나는데 관련 내용도 주석으로 달아뒀습니다. 맞게 수정 한 번 해보시는거 어떠세용?

  • 그리고 구현하진 모든 ViewController에 대해서 ViewModel 연동하기 전, 저희가 맞춘 BaseViewController에 맞춰 주시고 공통 메서드를 활용해보시는건 어떨까요?

  • layout 설정 또한 Autolayout에서 snapkit으로 변경 하는게 좋지 않을까 싶습니다! ㅎㅎ

바쁘시더라도 위 내용대로 한 번 리펙토링 진행 해보시겠어요 서현님? 좀 힘들다 하시면 힘든 부분 말씀해주시면 제가 할 수 있는한 최대한 많이 도와드리겠습니당
뭔가 제가 직접 변경하는것보다 서현님께서 해보시는게 더 좋을것같아서 일단 적어봅니다.
위 단계로 리펙토링 진행하면 그 다음은 ViewModel 활용 -> RxSwift 적용 순으로 해봐요

@cestbonciel
Copy link
Copy Markdown
Member Author

첨언해주신 부분들 보고 이번주 안으로 꼭 다시 수정해보겠습니다!
성훈님, 릴리 감사해요

Copy link
Copy Markdown
Member Author

@cestbonciel cestbonciel left a comment

Choose a reason for hiding this comment

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

/// signUpAction인 `SignUpViewModel`를 선언하는 과정이 없어서 해당 버튼을 클릭하면 Fatal error가 발생합니다. `static` 키워드를 활용한 `create` 메서드를 생성해서 해당 메서드에서 `ViewModel`을 주입하는 방식으로 구현 해주세용 
 @objc func moveNextView() {
        signUpAction.viewAction.moveView()
    }

넵, 확인하고 적용해보겠습니다

@Jeon0976 Jeon0976 closed this Apr 30, 2024
@Jeon0976 Jeon0976 deleted the Feature/login branch April 30, 2024 02:19
@Jeon0976 Jeon0976 restored the Feature/login branch May 7, 2024 03:39
@Jeon0976 Jeon0976 reopened this May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants