feat: 1.0.8 버전 업데이트에 따른 기능 추가 및 수정 by stealmh · Pull Request #175 · YAPP-Github/Pokit-iOS

stealmh · 2024-12-30T08:58:48Z

#️⃣연관된 이슈

📝작업 내용

편집하기 기능 추가
- 카테고리 추가 / 링크 삭제
버전 체크 로직 추가
미분류 링크 이동 및 삭제 API 구현
Perception 오류 수정 @ShapeKim98 짱

스크린샷 (선택)


`포킷이동 예시`	`포킷 추가하기 & 에러처리`	`미분류링크삭제`	`버전 분기`

💬리뷰 요구사항(선택)

FeaturePokit 필터 버튼
기존에 content를 기준으로 조건 처리를 하다보니 카테고리 화면에서 최신순/이름순 등 필터 버튼이 보이지 않는 것을 확인했습니다.

if !store.contents.isEmpty {}

contents는 미분류 링크 목록을 말합니다.
초기 folderType은 포킷이기 때문에 카테고리에 관한 정보만 받아 오기 때문에 미분류(store.contents)는 초기에 항상 Empty일 수 밖에 없고 따라서 필터 버튼이 보이지 않았습니다.

/// 카테고리가 있을 때 `정렬` 추가
if store.folderType == .folder(.포킷) && store.categories != nil {
    PokitIconLTextLink(...)
}

/// 미분류 링크가 있을 때 `편집하기` 추가
if store.folderType == .folder(.미분류) && !store.contents.isEmpty {}

적절히 분기처리를 통해 화면 별 필터 타입을 수정했습니다.

PokitNavigationBar Modifier 추가
기존에 Modifier에 closure를 통해 뷰를 적용했었습니다. 항상 반복되는 패턴으로 extension으로 navigationBar를 분리한다면 사실 클로저가 의미가 없다고 생각했습니다.

func pokitNavigationBar<Header: View>(_ header: @autoclosure () -> Header) -> some View {
    modifier(PokitNavigationBarModifier(header: header))
}

다음과 같이 사용하게 추가해두었습니다.

struct SomeView: View {
    var body: some View {
        VStack {
            ...
        }
-      .pokitNavigationBar {
-          navigationBar
-       }
+     .pokitNavigationBar(navigationBar)
    }
} 
extension SomeView {
    var navigationBar: some View { ... }
}

PokitSelect PokitSelectSheet 중복코드
기존에 있던 PokitSelect가 활용되는 피쳐가 생겨 기능 수정이 이루어졌습니다. (짱짱!) @ShapeKim98
버전체크 로직 추가
기존 SplashFeature에 해당 기능이 추가되었습니다. 스플래시 뷰 이후 버전업데이트가 필요하다면 강제 alert으로 앱스토어로 이동되게끔 적용했습니다. 로직의 경우 major와 minor가 바뀌었을 때만 앱스토어로 이동되는 alert이 나타나게 됩니다. 맨 뒷자리는 신경쓰지 않습니다.
혹시몰라 테스트 케이스도 작성해두었습니다.

// SplashFeature
case .onAppear:
reutrn .run { ...
    try await self.clock.sleep(for: .milliseconds(2000))
    /// Version Check
    let response = try await versionClient.버전체크().toDomain()
    guard
        let info = Bundle.main.infoDictionary,
        let currentVersion = info["CFBundleShortVersionString"] as? String else { return }
        let appStoreVersion = response
        let nowVersion = Version(currentVersion, trackId: response.trackId)
                
        if nowVersion < appStoreVersion {
            await send(.inner(.앱스토어_알림_활성화(trackId: response.trackId)))
            /// 비교를 통해 앱스토어 버전이 더 크다면 아래 로직을 타지 못하게 리턴
            return
        }
 }

etc
크리스마스 앱 아이콘도 내려야 하는데 PR리뷰하면서 고칠 때 같이 해버리겠읍니다.
fullScreenCover로 적용한 이유도 같이 주석 달아놨으니 참고하면 좋을듯!
편집하기 로직이 귀찮아도 간단해서 PR내용에는 상세하게 적지는 않았습니다. 바로 보고 이해하실 것 같음.

편집할 링크가 없을 시 바로 로직 종료하게 수정했습니다. (수정완)

close #173

ShapeKim98

수고하셨습니다! 특히 Perception 경고는 진짜.. 힘들었을 것 같네요.. 딱히 문제는 없고 시간 나실때 자잘한 코드 수정만 해주세여(크리스마스 푸키 내리는거 너무 슬픈데..)

ShapeKim98 · 2024-12-31T04:10:12Z

Projects/CoreKit/Sources/Data/Network/Version/VersionEndpoint.swift

-        return Constants.serverURL.appendingPathComponent(Constants.versionPath, conformingTo: .url)
+        let bundleID = Bundle.main.bundleIdentifier ?? ""
+        let countryCode = Locale.current.language.region?.identifier ?? ""
+        return URL(string: "https://itunes.apple.com/lookup?bundleId=\(bundleID)&country=\(countryCode)")!


저희 서버에 버전 체크 api 있지 않나여?

써봤는데 최신 버전이랍시고 1.0.1 리턴되던데 버려진 api느낌같아서 만들었습니다

써봤는데 최신 버전이랍시고 1.0.1 리턴되던데 버려진 api느낌같아서 만들었습니다

어라..ㅋㅋㅋㅋㅋㅋㅋㅋ

ShapeKim98 · 2024-12-31T06:41:38Z

Projects/Feature/FeatureLogin/Sources/Splash/SplashFeature.swift

    }
    /// - Scope Effect
    func handleScopeAction(_ action: Action.ScopeAction, state: inout State) -> Effect<Action> {
+        switch action {


Dependecies에 기본 제공 Dependency values에 openURL이 있긴한데 이걸 쓰시는건 어떤가유

죠습니다

수정: b4b9073

ShapeKim98 · 2024-12-31T06:42:35Z

Projects/Feature/FeatureLogin/Sources/Splash/SplashFeature.swift

+        case .alert:
+            return .none
+        }
        return .none


불필요 코드 제거 부탁합니당

수정: 7b44d7f

ShapeKim98 · 2024-12-31T06:44:08Z

Projects/Feature/FeatureLogin/Sources/Splash/SplashFeature.swift

        return .none
    }
+
+    func versionCheck() -> Effect<Action> {


이거 따로 쓰이는 곳이 있나요?

따로 빼려나가 까먹고 방치된듯 지우겠읍니다

수정: 7b44d7f

ShapeKim98 · 2024-12-31T06:45:54Z

Projects/DSKit/Sources/Components/PokitSelectSheet.swift

+import Util
+
+
+public struct PokitSelectSheet<Item: PokitSelectItem>: View {


시트가 PokitSelect에 너무 종속된건 아닌가 생각했었는데.. 확실히 빼는게 좋은거 같아요!

ShapeKim98 · 2024-12-31T06:57:05Z

Projects/Feature/FeaturePokit/Sources/PokitLinkEditView.swift

+        .padding(.top, 8)
+    }
+
+    var contentList: some View {


뷰를 내부 클로저로 빼는거랑 body에 작성하는거랑 차이를 모르겠어요.. 진짜 그냥 Perception버그 인거 같기도...

저도 아직도 모르겠음 더 찾아봐야겠습니다

ShapeKim98 · 2024-12-31T06:59:11Z

Projects/Feature/FeaturePokitTests/Sources/FeaturePokitTests.swift

 @testable import FeaturePokit
+@testable import Domain

 final class FeaturePokitTests: XCTestCase {


헉.. 테스트!

도메인이나 코어킷 테스트모듈도 필요할듯합니다..ㅠ 여기서 버전체크를 테스트 하고있음🥲

- 초기 folderType은 포킷이기 때문에 카테고리에 관한 정보만 받아온다. 따라서 미분류(store.contents)는 초기에 항상 Empty이기 때문에 필터 버튼이 보이지 않았음.

화면 전환이 되는 0.7초의 시간동안 버튼이 전환 되었음에도 이전 폴더 타입의 버튼액션을 받을 수 있었던 버그 수정

- 체크했을 때 id값 구분으로 체크/체크해제 구성 - 이전 화면에서 모든 미분류 링크를 받기 위해 생성자로 구성

- 포킷이동 버튼을 누르면 카테고리 선택 시트가 나오게 적용

- 편집하기 후 돌아왔을 때 최신 미분류 링크들을 새로 받는 로직 추가 - 항상 최신순 정렬

- 에러로직 추가 - 삭제하기 바텀시트 구성 - 편집할 링크가 없을 때 보여줄 화면 분기처리

stealmh added Feat 기능구현 Design 🎨 디자인 작업 Fix 기능 수정 API 🛜 기능구현 - API labels Dec 30, 2024

stealmh requested a review from ShapeKim98 December 30, 2024 08:58

stealmh self-assigned this Dec 30, 2024

stealmh linked an issue Dec 30, 2024 that may be closed by this pull request

1.0.8 버전 업데이트에 따른 기능 추가 및 수정 #173

Closed

5 tasks

ShapeKim98 approved these changes Dec 31, 2024

View reviewed changes

stealmh and others added 22 commits January 1, 2025 18:42

[fix] #173 카테고리 타입 별 버튼 수정

60a1018

- 초기 folderType은 포킷이기 때문에 카테고리에 관한 정보만 받아온다. 따라서 미분류(store.contents)는 초기에 항상 Empty이기 때문에 필터 버튼이 보이지 않았음.

[fix] #173 애니메이션 버그 수정

708300b

화면 전환이 되는 0.7초의 시간동안 버튼이 전환 되었음에도 이전 폴더 타입의 버튼액션을 받을 수 있었던 버그 수정

[feat] #173 편집하기 로직 연결

35aac43

[fix] #173 편집하기 조건 수정

67c1f36

[feat] navigationBar View Modifier autoclosure 버전 추가

00a1372

[fix] 미분류 카테고리 편집에서 card action 받지 못하게 수정

8460670

[feat] #173 편집하기 버튼 액션 추가

d090866

[feat] #173 편집하기 체크기능 및 화면리스트 구성

1c7cb18

- 체크했을 때 id값 구분으로 체크/체크해제 구성 - 이전 화면에서 모든 미분류 링크를 받기 위해 생성자로 구성

[feat] #173 편집하기 뷰 구성

53706b3

[feat] 버전업

ec08b3b

[feat] #173 뒤로가기 액션 추가

dd7c0ce

[feat] #173 편집하기 float button 구성

4b97cde

[fix] print 제거

c9b67c0

[fix] 레이아웃 수정

426a251

[feat] #173 Float Button Delegate 연결

0671f7d

[feat] #173 카테고리 추가 시트 구성

5e682a7

[feat] #173 포킷이동을 위한 시트 연결

f168d3a

- 포킷이동 버튼을 누르면 카테고리 선택 시트가 나오게 적용

[feat] #173 미분류 링크 이동 로직 추가

19ec4cd

[feat] #173 미분류 링크 삭제 API 추가

91b37b4

[feat] #173 미분류 로직 수정

f3a36ad

- 편집하기 후 돌아왔을 때 최신 미분류 링크들을 새로 받는 로직 추가 - 항상 최신순 정렬

[feat] #173 편집하기 로직 구성

9331748

- 에러로직 추가 - 삭제하기 바텀시트 구성 - 편집할 링크가 없을 때 보여줄 화면 분기처리

[refactor] #173 PokitSelect에 PokitSelectSheet 적용

59911dd

stealmh and others added 11 commits January 1, 2025 18:42

[fix] Perception Error 수정

82935a1

[fix] #173 Perception Error 수정 @ShapeKim98

088fa68

[feat] #173 버전 체크 로직 추가

fc0cdcb

[style] 컨벤션 수정

6b81488

[feat] 앱 아이콘 수정

a014b08

[feat] #173 편집할 링크가 없을 시 종료 로직 추가

5fa1b2c

[fix] 링크 비어있을 때 로직 종료 수정

735db31

[refactor] #173 링크 편집 개선

e17b3ac

[feat] #173 버전 체크 검증 테스트 케이스 추가

0f60b85

[fix] 불필요 코드 삭제

e884ac6

[feat] Dependency openURL 리팩토링

89594ce

stealmh force-pushed the feat/#170-link-categorize branch from b4b9073 to 89594ce Compare January 1, 2025 09:45

stealmh merged commit 6dc892d into develop Jan 1, 2025
1 check failed

stealmh deleted the feat/#170-link-categorize branch January 4, 2025 03:09

		import Util


		public struct PokitSelectSheet<Item: PokitSelectItem>: View {

Conversation

stealmh commented Dec 30, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

#️⃣연관된 이슈

📝작업 내용

스크린샷 (선택)

💬리뷰 요구사항(선택)

Uh oh!

ShapeKim98 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!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

stealmh commented Dec 30, 2024 •

edited

Loading