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

[2.3.0] 공지 제목 번역 기능 (실험실) #232

Merged
merged 23 commits into from
Sep 12, 2024

Conversation

lgvv
Copy link
Member

@lgvv lgvv commented Aug 27, 2024

내용

  • 쿠링에 공지 제목 번역 기능 도입

기대 효과

  • 외국인 학생도 이용할 수 있음.
  • 쿠링 AI를 활용한 미래지향적인 서비스로 변화
  • 실험실 활성화

개발 타겟

  • iOS 17.4 이상

개발 환경에 따른 한계

  • iOS 18.0 이상은 Translation Framework를 통해 시스템 UI 외에 더 유연하게 사용 가능.
    • 단, 18.0 이상 지원을 위해 현재((24.08.27) 기준으로 Xcode 16.0 beta 버전만 존재 (릴리즈 버전 없음)
    • 베타 버전의 경우 앱스토어 심사가 불가능.
    • 이에 따라 17.4 타겟으로 개발
    • 18.0 출시되면 텍스트만 한국어로 번역하는 Session을 변수로 선언하여 별도의 텍트스로 사용 가능
      • WWDC24 Translation 참고.

개발 과정에서 해결하기 어렵던 버그

  • SwipeAction에 번역 아이콘을 숨기는 형태로 개발했으나 상태가 바뀌어도 시스템 UI가 나타나지 않는 현상
  • SwiftUI로만 작성한 코드에서는 동작했으나 현재 TCA기반 프로젝트에서 미동작
    • TCA 가 원인인지는 모르겠음.
  • 따라서 UI 위에 버튼을 직접 노출하는 형태로 구현

동작영상

ScreenRecording_08-27-2024.13-43-48_1.MP4

SwipeAction을 통한 동작

2024-08-28.12.05.49.mov

샘플코드

import SwiftUI
import Translation

struct ViewTranslationView: View {
    @State private var showTranslation = false

    var body: some View {
        List(0..<5) {
            Text("\($0)")
                .swipeActions(edge: .trailing, allowsFullSwipe: true) {
                    Button {
                        print("클릭")
                        showTranslation.toggle()
                    } label: {
                        Image(systemName: "plus")
                    }
                }
        }
        .listStyle(.plain)
        .translationPresentation(
            isPresented: $showTranslation,
            text: "테스트"
        )
    }
}

@lgvv lgvv self-assigned this Aug 27, 2024
@lgvv lgvv added PBM Package Based Modularization (패키지 기반 모듈화) 외부 버그 외부 의존성 / 언어 / IDE 관련 버그 실험실 쿠링 실험실 기능, 실험실에 들어갈 베타 기능 and removed PBM Package Based Modularization (패키지 기반 모듈화) labels Aug 27, 2024
sunshiningsoo
sunshiningsoo previously approved these changes Aug 28, 2024
Copy link
Member

@sunshiningsoo sunshiningsoo left a comment

Choose a reason for hiding this comment

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

🫡🙌👍 짱입니다

@lgvv
Copy link
Member Author

lgvv commented Aug 28, 2024

/쿠링 패키지 빌드

Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

Copy link

📬 패키지 빌드에 실패했습니다. Actions에서 실패 결과를 확인해주세요.

sunshiningsoo
sunshiningsoo previously approved these changes Aug 28, 2024
@ku-ring ku-ring deleted a comment from github-actions bot Sep 11, 2024
Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

Copy link

📬 패키지 빌드에 실패했습니다. Actions에서 실패 결과를 확인해주세요.

@ku-ring ku-ring deleted a comment from github-actions bot Sep 11, 2024
@sunshiningsoo
Copy link
Member

/쿠링 패키지 빌드

Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

Copy link

📬 패키지 빌드에 실패했습니다. Actions에서 실패 결과를 확인해주세요.

@lgvv
Copy link
Member Author

lgvv commented Sep 11, 2024

/쿠링 패키지 빌드

@ku-ring ku-ring deleted a comment from github-actions bot Sep 11, 2024
@ku-ring ku-ring deleted a comment from github-actions bot Sep 11, 2024
@lgvv
Copy link
Member Author

lgvv commented Sep 11, 2024

/쿠링 머지

Copy link

🗿 머지 시도 실패. 승인 개수, 테스트 성공 여부 등 자동 머지 조건을 만족하는지 확인하세요.

@ku-ring ku-ring deleted a comment from github-actions bot Sep 11, 2024
@ku-ring ku-ring deleted a comment from github-actions bot Sep 11, 2024
# Conflicts:
#	package-kuring/Package.resolved
@lgvv
Copy link
Member Author

lgvv commented Sep 11, 2024

/쿠링 패키지 빌드

Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

Copy link

📬 패키지 빌드에 실패했습니다. Actions에서 실패 결과를 확인해주세요.

@sunshiningsoo
Copy link
Member

/쿠링 패키지 빌드

Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

Copy link

✅ 패키지를 성공적으로 빌드했습니다.

@lgvv
Copy link
Member Author

lgvv commented Sep 12, 2024

/쿠링 머지

Copy link

🗿 머지 시도 실패. 승인 개수, 테스트 성공 여부 등 자동 머지 조건을 만족하는지 확인하세요.

@lgvv
Copy link
Member Author

lgvv commented Sep 12, 2024

/쿠링 머지

@github-actions github-actions bot merged commit d62978c into develop/2.0.0 Sep 12, 2024
Copy link

🕊️ PR #232이 머지되었습니다.

@github-actions github-actions bot deleted the feature/#231_translation branch September 12, 2024 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. 실험실 쿠링 실험실 기능, 실험실에 들어갈 베타 기능 외부 버그 외부 의존성 / 언어 / IDE 관련 버그
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants