Feature/splash by Roy-wonji · Pull Request #3 · SpartCodig-iOS/MovieBooking

Roy-wonji · 2025-10-14T03:20:03Z

🔗 관련 이슈

관련 이슈: MovieDetailCoordinator의 child로 연결 #12 ✨[feat]: tabbar 구현 #13

✨ 작업 내용

스플래쉬 화면 구현
화면 진입 코드 rootview 구현
로그인 코디네이터 및 , 로그인 화면 기본 골격 구현

📸 Showcase

Close Issues 🔒 (닫을 Issue)

Close #1, #2

minneee

수고하셨습니다~

minneee · 2025-10-14T08:19:34Z

MovieBooking/Config/Realse.xcconfig

+
+OTHER_SWIFT_FLAGS[config=PROD][sdk=*] = $(inherited) -PROD
+SWIFT_ACTIVE_COMPILATION_CONDITIONS = PROD
+BUNDLE_DISPLAY_NAME = 책시리즈


책 시리즈는 뭔가요??

잘못 들어간거여서 제거 했습니다

minneee · 2025-10-14T08:21:27Z

MovieBooking/DesignSystem/Color/Colors.swift

+  static var megaboxPrimary: Color { .init(hex: "503396") }
+  static var megaboxViolet: Color { .init(hex: "9333EA") }


megabox 클론 앱은 아니니까 megabox 대신 다른 명칭을 쓰는건 어떤가요??

minneee · 2025-10-14T08:27:41Z

MovieBooking/App/Di/DIRegistry.swift

+
+  ///  Repository 등록
+  static func registerRepositories() async {
+    let repositories: [ Module] = [


띄어쓰기 수정해주세요~

Peter1119

고려해볼사항이 있어서
의견 나눠 보거나 혹은 수정 되면 다시 말씀해주세요 ~!

Peter1119 · 2025-10-14T11:23:01Z

.github/.gitMessage.md

+🪛[chore]: 코드 수정, 내부 파일 수정. </br>
+✨[feat]: 새로운 기능 구현. </br>
+🎨[style]: 스타일 관련 기능.(코드의 구조/형태 개선) </br>
+➕[add]: Feat 이외의 부수적인 코드 추가, 라이브러리 추가 </br>
+🔧[file]: 새로운 파일 생성, 삭제 시 </br>
+🐛[fix]: 버그, 오류 해결. </br>
+🔥[del]: 쓸모없는 코드/파일 삭제. </br>
+📝[docs]: README나 WIKI 등의 문서 개정. </br>
+💄[mod]: storyboard 파일,UI 수정한 경우. </br>
+✏️[correct]: 주로 문법의 오류나 타입의 변경, 이름 변경 등에 사용합니다. </br>
+🚚[move]: 프로젝트 내 파일이나 코드(리소스)의 이동. </br>
+⏪️[rename]: 파일 이름 변경이 있을 때 사용합니다. </br>
+⚡️[improve]: 향상이 있을 때 사용합니다. </br>
+♻️[refactor]: 전면 수정이 있을 때 사용합니다. </br>
+🔀[merge]: 다른브렌치를 merge 할 때 사용합니다. </br>
+✅ [test]: 테스트 코드를 작성할 때 사용합니다. </br>


이거 이전부터 궁금했는데
이건 자동으로 불러올 수 있는건가요 ?
아니면 가이드 같은건가요 ?

@Peter1119 가이드 입니다 불러올수도 있는데 제가 예전에 했는데 그건 나중에 설명 드릴께요 제 fork 에서는 자동으로 되게 설정 해 놓아서요!

Peter1119 · 2025-10-14T11:24:13Z

MovieBooking.xcodeproj/project.pbxproj

 /* Begin PBXBuildFile section */
+		7F6F8B432E9D27850046FA0E /* Auth in Frameworks */ = {isa = PBXBuildFile; productRef = 7F6F8B422E9D27850046FA0E /* Auth */; };
+		7F6F8B452E9D27850046FA0E /* Supabase in Frameworks */ = {isa = PBXBuildFile; productRef = 7F6F8B442E9D27850046FA0E /* Supabase */; };
+		7F6F8B482E9D279B0046FA0E /* WeaveDI in Frameworks */ = {isa = PBXBuildFile; productRef = 7F6F8B472E9D279B0046FA0E /* WeaveDI */; };


저희 DI 넣는거 고민한번 해보면 좋을 것 같아요
단일 레포라 꼭 필요할까 싶은 생각이 있습니다 ~!
이건 의견 나눠봐도 좋을 것 같아요 ~!

@Peter1119 어차피 단일 레포여도 그렇긴 한데 클린아키텍쳐 및 그걸 생각 했을때는 저는 di넣어도 될거 같은데 같이 의견 나누어봐요!

@Peter1119 전 결합도는 낮출수 있어서 좋을거 같습니다

Peter1119 · 2025-10-14T11:28:30Z

MovieBooking/DesignSystem/Color/Colors.swift

+public extension ShapeStyle where Self == Color {
+  static var basicPurple: Color { .init(hex: "503396") }
+  static var violet: Color { .init(hex: "9333EA") }
+  static var brightYellow: Color { .init(hex: "FEE500") }
+}


이전에 리뷰 남겼었는지 궁금한데
이렇게 넣으면 다크모드 대응은 어떻게 하나요 ?
뷰에서 감지해서 색상을 설정해줘야하나요 ?

만일 그렇다면 이미지 에셋이 더 편리하지 않은가 싶은 생각입니다 ~!

전 이미지 에셋이 무겁다고 느끼고 저희가 지금은 다크 모드는 대응을 안하닌 괜찬을거 같습니다
다크모드 일경우도 저 생삭 같은경우는 다크 모드 까지는 필요없다고 느낌니다 @Peter1119

Peter1119 · 2025-10-14T11:29:46Z

MovieBooking/DesignSystem/Font/PretendardFont.swift

+struct PretendardFont: ViewModifier {
+  public let family: PretendardFontFamily
+  public let size: CGFloat
+
+  public func body(content: Content) -> some View {
+    return content.font(.custom("PretendardVariable-\(family)", fixedSize: size))
+  }
+}


제가 알기론 modifier를 너무 많이 쓰면 성능상 좋지 않다고 해서
font의 extension으로만 참조해도 충분하지 않을까요 ?

Peter1119 · 2025-10-14T11:31:12Z

MovieBooking/Feature/Auth/Coordinator/Reducer/AuthCoordinator.swift

+  private func handleViewAction(
+    state: inout State,
+    action: View
+  ) -> Effect<Action> {
+    switch action {
+
+    }
+  }
+
+  private func handleAsyncAction(
+    state: inout State,
+    action: AsyncAction
+  ) -> Effect<Action> {
+    switch action {
+
+    }
+  }
+
+  private func handleNavigationAction(
+    state: inout State,
+    action: NavigationAction
+  ) -> Effect<Action> {
+    switch action {
+
+    }
+  }
+
+  private func handleInnerAction(
+    state: inout State,
+    action: InnerAction
+  ) -> Effect<Action> {
+    switch action {
+
+    }
+  }


우선 사용하지 않는다면 없어지는것도 깔끔할 것 같습니다 ~! 필요할때 추가하는 방식으로요 !

Peter1119 · 2025-10-14T11:35:20Z

MovieBooking/Feature/Splash/View/SplashView.swift

+import ComposableArchitecture
+
+struct SplashView: View {
+  @Bindable var store: StoreOf<Splash>


혹시 binding 되는 상태값이 없는 것 같은데 그럼 bindable하지 않아도 되지 않을까요 ?

Peter1119 · 2025-10-14T11:35:42Z

MovieBooking/Feature/Splash/View/SplashView.swift

+      .opacity(store.fadeOut ? 0.0 : 1.0)
+      .animation(.easeInOut(duration: 1), value: store.fadeOut)
+      .onAppear {
+        store.send(.view(.onAppear))


view action 정의 되어있는데 처리되지 않고 있는 것 같습니다 ~!

@Peter1119 사용 되고 있습니다

https://mehmetbaykar.com/posts/tca-tips-the-power-of-viewactions/

여기 링크 확인해보시겠어요 ?

@Peter1119 감사합니다 보일러 플레이트를 줄일수 있군요!

Peter1119 · 2025-10-14T11:37:59Z

MovieBooking/Feature/Splash/Reducer/Splash.swift

+  @CasePathable
+  public enum View : Equatable{
+    case onAppear
+    case startAnimationSequence


startAnimation
이렇게 간단하게 해도 의미는 통할 것 같아요 ~!

Peter1119 · 2025-10-14T11:39:27Z

MovieBooking/Feature/Splash/Reducer/Splash.swift

+
+
+@Reducer
+public struct Splash {


네이밍에 대한 생각을 같이 해보면 좋을 것 같은데
Splash, Login 과 같이 이름을 가져가면
Entity 정의할때 겹칠 것같아요
Movies, MovieList, MovieDetail 처럼요 ~!

저는 뒤에 Feature를 붙히는 것을 선호하는데 어떻게 생각하시나요 ?

@Peter1119 저도 처음에는 feature 를 쓰었는데 reducer 로 통일 해도 괘찮을거 같습니다 이건 한번 의견을 나누어 보면 좋을거 같아요!

Peter1119 · 2025-10-14T11:45:54Z

MovieBooking/App/Reducer/AppReducer.swift

+@Reducer
+struct AppReducer {
+
+  @ObservableState
+  enum State {
+    case splash(Splash.State)
+    case auth(AuthCoordinator.State)
+
+
+
+    init() {
+      self = .splash(.init())
+    }
+  }
+
+  enum Action: ViewAction {
+    case view(View)
+    case scope(ScopeAction)
+  }


App도 Reducer가 아닌 AppCoordinator로 할 수 있지 않을까요 ??

@Peter1119 흠 저건 좀 고민 해볼께요! 전 reeucer 하는게 더 낳다고 생각 이들어서요!
어자피 각 씬에 코디네이터가 있는데 굳이 앱까지 필요할까라고 생각듭니다

Peter1119

고생하셨습니다 ~!

Roy-wonji added 12 commits October 14, 2025 10:58

[feat]: di 설정 #1

77e2cc1

[feat]: application 구현 #1

93acc19

[feat]: 슈퍼베이스 과련 config 추가 #1

c9a5eba

[feat]: 디자인 시스템 컬러 및 폰트 구현 #1

f1ac46c

[feat]: splash 뷰 및 reducer 구현 #1

8e85359

[feat]: 주석 삭제 #1

43687d9

[feat]: 컬러 추가 #1

ad3871e

[chore]: 스플래쉬 action 관련 코드 수정 #1

e27c5ad

[chore]: 라이브러리 관련 수정 #1

f7809e5

[feat]: auth 관련 화면 추가 및 기본 설정 구현 #2

eae5de3

[feat]: 앱시작시 화면 추가 구현 및 리듀서 구현 #2

cbc67ce

[feat]: pr , issue templete 추가

65d81f7

Roy-wonji requested review from Peter1119 and minneee October 14, 2025 03:20

Roy-wonji self-assigned this Oct 14, 2025

Roy-wonji added ✨ 기능추가 새로운 기능 추가 🎨 디자인 UI 디자인 작업 labels Oct 14, 2025

[chore]: delay 시간 변경 #1

4de328c

minneee reviewed Oct 14, 2025

View reviewed changes

Roy-wonji added 2 commits October 14, 2025 18:59

[chore]: pr 리뷰 수정

31eb40e

[chore]: 코드 변경에 따른 코드 수정 #2

7556f26

Peter1119 reviewed Oct 14, 2025

View reviewed changes

Roy-wonji added 2 commits October 14, 2025 21:13

[chore]: 코드 변경에 따른 코드 수정 #2

56f1e36

[chore]: 코드 변경에 따른 코드 수정 #2

81f4ceb

Peter1119 self-requested a review October 15, 2025 02:43

Peter1119 approved these changes Oct 15, 2025

View reviewed changes

minneee approved these changes Oct 15, 2025

View reviewed changes

Roy-wonji merged commit 98306c7 into develop Oct 15, 2025

Roy-wonji deleted the feature/splash branch October 15, 2025 02:53

		static var megaboxPrimary: Color { .init(hex: "503396") }
		static var megaboxViolet: Color { .init(hex: "9333EA") }

Conversation

Roy-wonji commented Oct 14, 2025

🔗 관련 이슈

✨ 작업 내용

📸 Showcase

Close Issues 🔒 (닫을 Issue)

Uh oh!

minneee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Peter1119 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Roy-wonji Oct 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Roy-wonji Oct 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Peter1119 left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Roy-wonji Oct 14, 2025 •

edited

Loading

Roy-wonji Oct 14, 2025 •

edited

Loading