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.

Sign up for GitHub

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

Jump to bottom

[Release] 2.0.0 - RC #205

Merged
merged 16 commits into from
May 17, 2024
Merged

[Release] 2.0.0 - RC #205

merged 16 commits into from
May 17, 2024

Conversation

lgvv
Copy link
Member

@lgvv lgvv commented Apr 30, 2024

내용

  • 릴리즈 전 불필요한 파일 삭제
  • 릴리즈 전 최종 개발 코드 후보
    • 크리티컬하지 않은 잔여 이슈는 해당 버전 이후에 대응
  • 릴리즈 예정일 5월 첫주

iOS: @x-0o0 @wonniiii
Android: @mwy3055 @l2hyunwoo
Server: @zbqmgldjfh

@lgvv lgvv requested a review from wonniiii April 30, 2024 14:00
@lgvv lgvv self-assigned this Apr 30, 2024
@x-0o0
Copy link
Contributor

x-0o0 commented Apr 30, 2024

앱 버전 올려주세요

@lgvv
Copy link
Member Author

lgvv commented Apr 30, 2024

앱 버전 올려주세요

버전 2.0.0으로 세팅되어 있는데 혹시 어떤 버전일까요??

@x-0o0
Copy link
Contributor

x-0o0 commented Apr 30, 2024 •
edited
Loading

앱 버전 올려주세요

버전 2.0.0으로 세팅되어 있는데 혹시 어떤 버전일까요??

아 빌드버전이요

혹시 QA 거치고 수정 가능성이 있는 건가요? 그러면 빌드버전은 제가 테플 올리는 시간으로 맞추고
정식 릴리즈 될 버전이면 건우님기준으로 버저닝 해주세요.

@x-0o0
Copy link
Contributor

x-0o0 commented Apr 30, 2024

/쿠링 패키지 빌드하고 패키지 테스트

@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

🔨 iOS 17 iPhone 15 Pro 에서 쿠링 패키지를 테스트 합니다.

@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

📬 iOS17 에서 패키지 테스트에 실패했습니다. Actions에서 실패 결과를 확인해주세요.

@x-0o0
Copy link
Contributor

x-0o0 commented May 1, 2024 •
edited
Loading

@lgvv

스크린샷 2024-05-01 오후 5 35 50

Value of type 'KuringLink' has no member 'registerAuthorization'

빌드가 안되구 있어요!

원인 파악에 혹시 문제가 있다면 멘션주세요.

-> 에러 원인은 Package 에서 PushNotifications 타겟의 디펜던시 정의에 Dependencies 등 import 하는 모듈이 누락되어 있어서 그렇습니다~!

@x-0o0
Copy link
Contributor

x-0o0 commented May 1, 2024 •
edited
Loading

@lgvv

Merge branch 'feature/lgvv/auth' into release/2.0.0

이미 #204 통해서 develop 에 반영되어있어요~!

@x-0o0
Copy link
Contributor

x-0o0 commented May 1, 2024

/쿠링 패키지 빌드

@github-actions GitHub Actions
Copy link

github-actions bot commented May 1, 2024

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

@github-actions GitHub Actions
Copy link

github-actions bot commented May 1, 2024

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

yeon-kyu
yeon-kyu previously approved these changes May 1, 2024
View reviewed changes
Copy link
Member

@yeon-kyu yeon-kyu left a comment

Choose a reason for hiding this comment

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

구두로 설명해주신 부분들 이해했습니다. 고생많으셨습니다~!

@x-0o0 x-0o0 force-pushed the release/2.0.0 branch from cff483c to 79aaebe Compare May 1, 2024 17:52
@x-0o0
Copy link
Contributor

x-0o0 commented May 2, 2024

/쿠링 패키지 빌드

@github-actions GitHub Actions
Copy link

github-actions bot commented May 2, 2024

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

@github-actions GitHub Actions
Copy link

github-actions bot commented May 2, 2024

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

@sunshiningsoo
[수정] 앱아이콘 Alternate 리뷰 수정
3e8b8d3 
- 기존 디렉토리에 직접 들고 있는 png 파일 Asset의 AppIcon으로 변환해서 저장해두었습니다.
- plist 파일에서 앱 아이콘 변환하는 것을 pbx/general/ALTERNATE_APPICON_NAMES로 변경합니다.
@lgvv
Copy link
Member Author

lgvv commented May 2, 2024 •
edited
Loading

/쿠링 패키지 빌드

@github-actions GitHub Actions
Copy link

github-actions bot commented May 2, 2024

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

@github-actions GitHub Actions
Copy link

github-actions bot commented May 2, 2024

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

@sunshiningsoo
Copy link
Member

@x-0o0
3e8b8d3 에서 수정했습니다~! 테플 업로드 한번 부탁드려요!

이전과의 변화로는 아래 두가지입니다

  • 디렉토리에서 직접 png 파일 들고 있는 것 -> Asset의 AppIcon으로 변경
  • plist에서 Alternate App Icon 관리 -> BuildSetting/Alternate App Icon Sets 에서 관리

@ku-ring ku-ring deleted a comment from mwy3055 May 2, 2024
@x-0o0
Copy link
Contributor

x-0o0 commented May 2, 2024

@x-0o0 3e8b8d3 에서 수정했습니다~! 테플 업로드 한번 부탁드려요!

이전과의 변화로는 아래 두가지입니다

  • 디렉토리에서 직접 png 파일 들고 있는 것 -> Asset의 AppIcon으로 변경
  • plist에서 Alternate App Icon 관리 -> BuildSetting/Alternate App Icon Sets 에서 관리

고생하셨습니다. 테플 빌드버전 1157 업로드 했고 이슈 메일은 없었습니다. 👍

@lgvv lgvv enabled auto-merge (squash) May 15, 2024 17:52
@lgvv
Copy link
Member Author

lgvv commented May 15, 2024

/쿠링 머지

@github-actions github-actions bot added the automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. label May 15, 2024
@github-actions GitHub Actions
Copy link

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

@lgvv lgvv requested a review from sunshiningsoo May 15, 2024 17:53
@lgvv lgvv disabled auto-merge May 15, 2024 17:53
sunshiningsoo
sunshiningsoo approved these changes May 17, 2024
View reviewed changes
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.

머지 후 다시 작업해요!

@sunshiningsoo
Copy link
Member

/쿠링 머지

@github-actions github-actions bot merged commit 3b8229d into develop/2.0.0 May 17, 2024
@github-actions github-actions bot deleted the release/2.0.0 branch May 17, 2024 03:08
@github-actions GitHub Actions
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@x-0o0 x-0o0 x-0o0 left review comments

@sunshiningsoo sunshiningsoo sunshiningsoo approved these changes

@yeon-kyu yeon-kyu yeon-kyu left review comments

@wonniiii wonniiii Awaiting requested review from wonniiii

Assignees

@lgvv lgvv

Labels
automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lgvv @x-0o0 @sunshiningsoo @yeon-kyu @wonniiii