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

[Chore] #676 - Fastlane release lane 수정 #677

Merged
merged 12 commits into from
Nov 1, 2022

Conversation

hyun99999
Copy link
Member

@hyun99999 hyun99999 commented Oct 30, 2022

🔥Pull requests

⛳️ 작업한 브랜치

👷 작업한 내용

  • release lane 에서 buid app 명령어의 provisioning profile 설정하는 파라미터가 없어서 설정해주었습니당
  • precheck 에서 In-app 세팅
  • IDFA 세팅

🚨참고 사항

  • 지금 버전을 터미널에서 명령어와 함께 올릴 수 있도록 했는데 그렇게 되지 않아서 프로젝트에서 수동으로 업데이트해서 진행했습니다. 테스트 플라이트를 설정할 때에도 빌드 번호가 올라가지 않아서 버전을 올리고 새로 테플을 등록햇더니 되어서 이번에도 그런 문제인가 싶어서 올려보았습니당.(뭔가 초기화를 처음에 해줘야하는 느낌쓰로)
  • 일단, 멀티타겟을 사용하면서 기본보다 필요한 옵션들이 많았으니까 다음 릴리즈 때도 lane 을 테스트해보면 좋을 듯 합니다.

release lane 을 적용하면서 두 가지 에러를 만났습니다.

❗️ error1

  • Precheck cannot check In-app purchases with the App Store Connect API Key (yet). Exclude In-app purchases from precheck, disable the precheck step in your build step, or use Apple ID login

프로젝트에서 in-app 결제가 없기 때문에 precheck 단계에서 in-app 결제를 제외하여 해결하였습니다.

1

  • precheck 단계는 심사에 제출하기 전 fastlane 에서 자체적으로 체크해주는 단계를 말합니다.(아래의 설명 참조)

2

upload_to_app_store(

    // ...
    // ✅ Should precheck check in-app purchases? 기본값은 true 이다.
    precheck_include_in_app_purchases: false
)

❗️error2 - IDFA

  • Use of Advertising Identifier (IDFA) is required to submit

3

IDFA(광고 식별자)를 사용하지 않기 때문에 submission_information 파라미터를 사용해서 IDFA 세팅을 포함해주었습니다.

upload_to_app_store(

    // ...

    // ✅ Use the submission_information parameter for additional submission specifiers, including compliance and IDFA settings.
    submission_information: { add_id_info_uses_idfa: false }
)

📟 관련 이슈

@hyun99999 hyun99999 added Chore 그 이외의 잡일/ 버전 코드 수정, 패키지 구조 변경, 파일 이동, 파일이름 변경 👼타락pOwEr천사현규 크로아서버 환영요 labels Oct 30, 2022
@hyun99999 hyun99999 self-assigned this Oct 30, 2022
Copy link
Member

@yangsubinn yangsubinn left a comment

Choose a reason for hiding this comment

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

🙇‍♀️👍🍎

@hyun99999 hyun99999 merged commit 9be5af5 into TeamSparker:release-1.0.4 Nov 1, 2022
@hyun99999 hyun99999 deleted the release-1.0.4/#676 branch November 1, 2022 04:36
Comment on lines -63 to +74
increment_version_number(version_number: options[:version])
build_app(workspace: "Spark-iOS.xcworkspace", scheme: "Spark-release")
increment_version_number(
version_number: options[:version],
xcodeproj: "Spark-iOS.xcodeproj"
)
build_app(
workspace: "Spark-iOS.xcworkspace",
scheme: "Spark-beta",
export_method: "app-store",
export_options: {
provisioningProfiles: {
"com.TeamSparker.Spark" => "Spark AppStore",
"com.TeamSparker.Spark.SparkNotificationService" => "SparkNotificationService AppStore"
}
Copy link
Member Author

Choose a reason for hiding this comment

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

@yangsubin
여기 였나봐요 🥲

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore 그 이외의 잡일/ 버전 코드 수정, 패키지 구조 변경, 파일 이동, 파일이름 변경 👼타락pOwEr천사현규 크로아서버 환영요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Chore] Fastlane release lane 수정
2 participants