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.0.0] NoticeFeatures 에 bookmarks 디펜던시 로직 적용 #87

Merged
merged 4 commits into from
Jan 7, 2024

Conversation

x-0o0
Copy link
Contributor

@x-0o0 x-0o0 commented Dec 30, 2023

  • 이슈: Package 쪽에 bookmarks.remove(_:) 가 제대로 되지 않는 이슈
  • 앱프로젝트에서는 잘 동작
북마크 전 공지리스트 북마크 전 공지 화면 북마크 후 공지 화면 북마크 후 공지리스트
Simulator Screenshot - iPhone 15 Pro - 2023-12-30 at 16 17 27 Simulator Screenshot - iPhone 15 Pro - 2023-12-30 at 16 17 30 Simulator Screenshot - iPhone 15 Pro - 2023-12-30 at 16 17 34 Simulator Screenshot - iPhone 15 Pro - 2023-12-30 at 16 17 37

- 이슈: bookmarks.remove(_:) 가 되지 않는 이슈
@x-0o0 x-0o0 requested a review from lgvv December 30, 2023 07:21
@x-0o0 x-0o0 marked this pull request as draft December 30, 2023 07:21
@x-0o0 x-0o0 self-assigned this Dec 30, 2023
@x-0o0 x-0o0 added 공지리스트 공지디테일, 공지리스트 기능 공지보관함 공지보관함(북마크) 기능 labels Dec 30, 2023
@x-0o0 x-0o0 marked this pull request as ready for review January 6, 2024 16:25
Copy link
Contributor Author

@x-0o0 x-0o0 left a comment

Choose a reason for hiding this comment

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

/쿠링 패키지 테스트 다시

Copy link
Contributor Author

@x-0o0 x-0o0 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 assigned x-0o0 and unassigned x-0o0 Jan 6, 2024
@x-0o0 x-0o0 force-pushed the feature/jaesung/notices-bookmarks-ui branch from 7cf04f9 to ddf2070 Compare January 6, 2024 17:10
@x-0o0
Copy link
Contributor Author

x-0o0 commented Jan 6, 2024

/쿠링 패키지 테스트

Copy link

github-actions bot commented Jan 6, 2024

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

Copy link

github-actions bot commented Jan 6, 2024

✅ iOS17 에서 패키지를 성공적으로 테스트 했습니다.

@x-0o0 x-0o0 linked an issue Jan 6, 2024 that may be closed by this pull request
@x-0o0 x-0o0 enabled auto-merge (squash) January 6, 2024 17:30
@lgvv
Copy link
Member

lgvv commented Jan 7, 2024

/쿠링 패키지 테스트

Copy link

github-actions bot commented Jan 7, 2024

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

Copy link

github-actions bot commented Jan 7, 2024

✅ iOS17 에서 패키지를 성공적으로 테스트 했습니다.

- `pull_request_review_comment` 이벤트 제거하고 actions/checkout@v4 에서 PR 브랜치 HEAD 로 체크하웃 하도록 수정
@x-0o0
Copy link
Contributor Author

x-0o0 commented Jan 7, 2024

/쿠링 패키지 테스트

Copy link

github-actions bot commented Jan 7, 2024

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

Copy link

github-actions bot commented Jan 7, 2024

✅ iOS17 에서 패키지를 성공적으로 테스트 했습니다.

@x-0o0 x-0o0 merged commit 0f21db2 into develop/2.0.0 Jan 7, 2024
@x-0o0 x-0o0 deleted the feature/jaesung/notices-bookmarks-ui branch January 7, 2024 12:11
@x-0o0 x-0o0 added the automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. label Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. 공지리스트 공지디테일, 공지리스트 기능 공지보관함 공지보관함(북마크) 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[2.0.0] NoticeFeatures 에 북마크 디펜던시 적용하기
2 participants