Merge pull request #163 from Team-Ampersand/154-modify-personnel-hide

🔀 :: [#154] 인원 수정 기능 숨겨두기
Team-Ampersand · Aug 13, 2023 · a6454d8 · a6454d8
2 parents aef6327 + 1a97129
commit a6454d8
Show file tree

Hide file tree

Showing 6 changed files with 57 additions and 49 deletions.
diff --git a/Projects/Feature/HomeFeature/Sources/Moordinator/HomeMoordinator.swift b/Projects/Feature/HomeFeature/Sources/Moordinator/HomeMoordinator.swift
@@ -32,6 +32,7 @@ final class HomeMoordinator: Moordinator {
         self.inputDialogFactory = inputDialogFactory
     }
 
+    // swiftlint: disable cyclomatic_complexity
     func route(to path: RoutePath) -> MoordinatorContributors {
         guard let path = path.asDotori else { return .none }
         switch path {
@@ -72,6 +73,7 @@ final class HomeMoordinator: Moordinator {
         }
         return .none
     }
+    // swiftlint: enable cyclomatic_complexity
 }
 
 private extension HomeMoordinator {

diff --git a/Projects/Feature/HomeFeature/Sources/Scene/Enums/MassageStatusType+button.swift b/Projects/Feature/HomeFeature/Sources/Scene/Enums/MassageStatusType+button.swift
@@ -5,9 +5,10 @@ import UserDomainInterface
 
 extension MassageStatusType {
     func buttonDisplay(userRole: UserRoleType) -> String {
-        guard userRole == .member else {
-            return L10n.Home.modifyLimitButtonTitle
-        }
+        #warning("FIXME: 디자인 변경 대응 시 재활성화")
+//        guard userRole == .member else {
+//            return L10n.Home.modifyLimitButtonTitle
+//        }
         switch self {
         case .can: return L10n.Home.canMassageButtonTitle
         case .cant: return L10n.Home.cantApplyButtonTitle
@@ -16,9 +17,10 @@ extension MassageStatusType {
     }
 
     func buttonIsEnabled(userRole: UserRoleType) -> Bool {
-        guard userRole == .member else {
-            return true
-        }
+        #warning("FIXME: 디자인 변경 대응 시 재활성화")
+//        guard userRole == .member else {
+//            return true
+//        }
         switch self {
         case .can, .applied: return true
         case .cant: return false

diff --git a/Projects/Feature/HomeFeature/Sources/Scene/Enums/SelfStudyStatusType+button.swift b/Projects/Feature/HomeFeature/Sources/Scene/Enums/SelfStudyStatusType+button.swift
@@ -5,9 +5,6 @@ import UserDomainInterface
 
 extension SelfStudyStatusType {
     func buttonDisplay(userRole: UserRoleType) -> String {
-        guard userRole == .member else {
-            return L10n.Home.modifyLimitButtonTitle
-        }
         switch self {
         case .can: return L10n.Home.canSelfStudyButtonTitle
         case .applied: return L10n.Home.appliedSelfStudyButtonTitle
@@ -17,9 +14,6 @@ extension SelfStudyStatusType {
     }
 
     func buttonIsEnabled(userRole: UserRoleType) -> Bool {
-        guard userRole == .member else {
-            return true
-        }
         switch self {
         case .can, .applied: return true
         case .cant, .impossible: return false

diff --git a/Projects/Feature/HomeFeature/Sources/Scene/HomeViewController.swift b/Projects/Feature/HomeFeature/Sources/Scene/HomeViewController.swift
@@ -55,6 +55,7 @@ final class HomeViewController: BaseStoredViewController<HomeStore> {
         self.navigationItem.setRightBarButton(myInfoBarButtonItem, animated: true)
     }
 
+    // swiftlint: disable function_body_length
     override func bindAction() {
         viewDidLoadPublisher
             .map { Store.Action.viewDidLoad }
@@ -122,6 +123,9 @@ final class HomeViewController: BaseStoredViewController<HomeStore> {
             .store(in: &subscription)
     }
 
+    // swiftlint: enable function_body_length
+
+    // swiftlint: disable function_body_length
     override func bindState() {
         let sharedState = store.state.share()
             .receive(on: DispatchQueue.main)
@@ -222,4 +226,5 @@ final class HomeViewController: BaseStoredViewController<HomeStore> {
             .sink(receiveValue: massageApplicationCardView.updateRecentRefresh(date:))
             .store(in: &subscription)
     }
+    // swiftlint: enable function_body_length
 }
diff --git a/Projects/Feature/HomeFeature/Sources/Scene/Store/HomeStore.swift b/Projects/Feature/HomeFeature/Sources/Scene/Store/HomeStore.swift
@@ -201,24 +201,25 @@ private extension HomeStore {
     }
 
     func applySelfStudyButtonDidTap() {
-        guard currentState.currentUserRole == .member else {
-            let inputDialogRoutePath = DotoriRoutePath.inputDialog(
-                title: L10n.Home.selfStudyModifyLimitTitle,
-                placeholder: "\(currentState.selfStudyInfo.1)",
-                inputType: .number
-            ) { [modifySelfStudyPersonnelUseCase, weak self] limit in
-                do {
-                    guard let limitInt = Int(limit) else { return }
-                    try await modifySelfStudyPersonnelUseCase(limit: limitInt)
-                    await DotoriToast.makeToast(text: L10n.Home.completeToModifySelfStudyLimitTitle, style: .success)
-                    self?.send(.refreshSelfStudyButtonDidTap)
-                } catch {
-                    await DotoriToast.makeToast(text: error.localizedDescription, style: .error)
-                }
-            }
-            route.send(inputDialogRoutePath)
-            return
-        }
+        #warning("FIXME: 디자인 변경 대응 시 재활성화")
+//        guard currentState.currentUserRole == .member else {
+//            let inputDialogRoutePath = DotoriRoutePath.inputDialog(
+//                title: L10n.Home.selfStudyModifyLimitTitle,
+//                placeholder: "\(currentState.selfStudyInfo.1)",
+//                inputType: .number
+//            ) { [modifySelfStudyPersonnelUseCase, weak self] limit in
+//                do {
+//                    guard let limitInt = Int(limit) else { return }
+//                    try await modifySelfStudyPersonnelUseCase(limit: limitInt)
+//                    await DotoriToast.makeToast(text: L10n.Home.completeToModifySelfStudyLimitTitle, style: .success)
+//                    self?.send(.refreshSelfStudyButtonDidTap)
+//                } catch {
+//                    await DotoriToast.makeToast(text: error.localizedDescription, style: .error)
+//                }
+//            }
+//            route.send(inputDialogRoutePath)
+//            return
+//        }
 
         Task.catching {
             if self.currentState.selfStudyStatus == .applied {
@@ -241,24 +242,25 @@ private extension HomeStore {
     }
 
     func applyMassageButtonDidTap() {
-        guard currentState.currentUserRole == .member else {
-            let inputDialogRoutePath = DotoriRoutePath.inputDialog(
-                title: L10n.Home.massageModifyLimitTitle,
-                placeholder: "\(currentState.massageInfo.1)",
-                inputType: .number
-            ) { [modifyMassagePersonnelUseCase, weak self] limit in
-                do {
-                    guard let limitInt = Int(limit) else { return }
-                    try await modifyMassagePersonnelUseCase(limit: limitInt)
-                    await DotoriToast.makeToast(text: L10n.Home.completeToModifyMassageLimitTitle, style: .success)
-                    self?.send(.refreshMassageButtonDidTap)
-                } catch {
-                    await DotoriToast.makeToast(text: error.localizedDescription, style: .error)
-                }
-            }
-            route.send(inputDialogRoutePath)
-            return
-        }
+        #warning("FIXME: 디자인 변경 대응 시 재활성화")
+//        guard currentState.currentUserRole == .member else {
+//            let inputDialogRoutePath = DotoriRoutePath.inputDialog(
+//                title: L10n.Home.massageModifyLimitTitle,
+//                placeholder: "\(currentState.massageInfo.1)",
+//                inputType: .number
+//            ) { [modifyMassagePersonnelUseCase, weak self] limit in
+//                do {
+//                    guard let limitInt = Int(limit) else { return }
+//                    try await modifyMassagePersonnelUseCase(limit: limitInt)
+//                    await DotoriToast.makeToast(text: L10n.Home.completeToModifyMassageLimitTitle, style: .success)
+//                    self?.send(.refreshMassageButtonDidTap)
+//                } catch {
+//                    await DotoriToast.makeToast(text: error.localizedDescription, style: .error)
+//                }
+//            }
+//            route.send(inputDialogRoutePath)
+//            return
+//        }
 
         Task.catching {
             if self.currentState.massageStatus == .applied {

diff --git a/Scripts/.swiftlint.yml b/Scripts/.swiftlint.yml
@@ -3,6 +3,9 @@ excluded:
   - Dependencies
   - Carthage
 
+disabled_rules:
+  - void_function_in_ternary
+
 identifier_name:
   min_length: 1
   max_length: 60
@@ -21,4 +24,4 @@ line_length:
 type_name:
   min_length: 2
   max_length: 60
-  allowed_symbols: [_]
+  allowed_symbols: [_]