Skip to content

Conversation

@0Hooni
Copy link
Collaborator

@0Hooni 0Hooni commented Nov 21, 2024

🤔 배경

  • 공동 작업시 이를 중계해줄 이벤트 허브가 필요했다

📃 작업 내역

  • 공동 작업에 사용될 임시 이벤트 허브를 구현했습니다.
  • 이벤트 허브 내 단일 이벤트를 관리해줄 이벤트 매니저를 구현했습니다.
  • develop과 합치는 과정에서 리네임이 좀 많아졌습니다.

✅ 리뷰 노트

임시코드가 많아 넘어가는 부분으로 봐주세요

🎨 스크린샷

x

🚀 테스트 방법

x

0Hooni and others added 11 commits November 18, 2024 17:16
nil인 경우가 존재할 수 있음

Co-Authored-By: YeongHoon Song <37678646+0Hooni@users.noreply.github.com>
Co-Authored-By: YeongHoon Song <37678646+0Hooni@users.noreply.github.com>
Co-Authored-By: seuhong <66902876+hsw1920@users.noreply.github.com>
- 추후 Priority Queue로 최적화 예정

Co-Authored-By: YeongHoon Song <37678646+0Hooni@users.noreply.github.com>
Co-Authored-By: YeongHoon Song <37678646+0Hooni@users.noreply.github.com>
Co-Authored-By: seuhong <66902876+hsw1920@users.noreply.github.com>
Co-Authored-By: seuhong <66902876+hsw1920@users.noreply.github.com>
@0Hooni 0Hooni linked an issue Nov 21, 2024 that may be closed by this pull request
3 tasks
Copy link
Collaborator

@hsw1920 hsw1920 left a comment

Choose a reason for hiding this comment

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

추후 코드리뷰 필요.

@0Hooni 0Hooni merged commit 4fc9a70 into develop Nov 21, 2024
1 check passed
@0Hooni 0Hooni deleted the feat/#47-make-event-hub branch November 21, 2024 05:24
@youn9k youn9k added the ✨ feat 새로운 기능 추가 label Nov 21, 2024
let point: CGPoint
let size: CGSize
let scale: CGFloat
let owner: String
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

id: 구별하기 위해
nickname: 중복될 수 있음

추후 오너에 대한 정보를 struct로 변경 필요

@youn9k
Copy link
Member

youn9k commented Nov 23, 2024

라이브 코드 리뷰 잘봤습니다ㅎㅎ
이벤트 허브 쪽 코드들이 깔끔해서 가독성이 좋네요!

@Kiyoung-Kim-57
Copy link
Member

LGTM! 라이브 코드 리뷰 설명이 너무 좋아요

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

Labels

✨ feat 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

이벤트 허브를 구현한다

5 participants