Skip to content
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

[PI-43] 1차 배포 준비: 약속관리화면 연결, Bundle Id 변경 #57

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Jinsujin
Copy link
Contributor

@Jinsujin Jinsujin commented Apr 13, 2023

ISSUE


작업 내용

  • 약속 관리 화면을 메인에 연결
  • 약속관리 화면에서 처리하던 화면전환을 HomeContainer 가 처리하도록 변경
  • Bundle Id 변경: 배포시 에러가 나서 수정이 필요했습니다
  • https://jinsujin.github.io/ ad-hoc 배포

처리 못한 작업

  • 상세화면 연결: 코멘트 봐주세요

실행 화면

Screen Shot Screen Shot
0413-managementView

Check List

  • PR 제목은 [ISSUE-{N}] PR 제목으로 작성
  • CI/CD 통과 여부
  • 1명 이상의 Approve 확인 후 Merge
  • 관련 이슈 연결

@Jinsujin Jinsujin self-assigned this Apr 13, 2023
Comment on lines 144 to 157

case let .standbyDetail(state):
// TODO: 상세화면 띄우기
return .none

case let .confirmedDetail(state):
// TODO: 상세화면 띄우기
return .none
Copy link
Contributor Author

@Jinsujin Jinsujin Apr 13, 2023

Choose a reason for hiding this comment

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

상세화면 처리에 대해

약속 관리 화면에서 처리하던 상세화면을 상위 객체-HomeContainerCore 가 처리할 수 있도록 액션을 연결했습니다.

@GangWoon
선택한 Promise 에 대해 상세화면을 띄우는 로직이 이미 있는데, 이를 활용해야 할까요..?
아니면 약속관리에서 띄우는 상세화면에 대한 State 를 따로 만들어서 처리해도 괜찮을까요?

private extension HomeContainerCore.State {
    var selectedPromise: Promise? {
        guard
            let destinationState,
            let state = (/HomeContainerCore.DestinationState.promiseList).extract(from: destinationState),
            let promise = state.selectedPromise
        else { return nil }

        return promise
    }
if let selectedPromise = viewStore.selectedPromise {
    PromiseDetailView(state: .init(promise: selectedPromise))
        .opacity(viewStore.selectedDetent == .large ? 1 : .zero)
}

Copy link
Contributor

Choose a reason for hiding this comment

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

따로 관리하셨으면 좋겠습니다.
HomeContainer에서 사용되는 PromiseDetail의 경우 조금 UX를 제공합니다.(풀 스크린 오버가 아니며, 다른 화면도 같이 노출될 수 있는 환경입니다.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 알겠습니다

@Jinsujin Jinsujin changed the title [PI-43] 약속관리화면을 메인과 연결 [PI-43] 1차 배포 준비: 약속관리화면 연결, Bundle Id 변경 Apr 13, 2023
Copy link
Contributor

@junyng junyng left a comment

Choose a reason for hiding this comment

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

Adhoc 배포 방법에 대해 관심이 생기네요
빠르게 배포 적용해주셔서 감사합니다
모듈 명 관련해 코맨트 남겼습니다
수고많으셨습니다 🙇‍♂️

AppPackage/Package.swift Outdated Show resolved Hide resolved
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