[Feature] 화이트보드 입장 핸들링 #141
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
choijungp
approved these changes
Dec 3, 2024
| @@ -139,6 +142,7 @@ extension WhiteboardRepository: NearbyNetworkConnectionDelegate { | |||
| with context: Data?, | |||
| isHost: Bool | |||
| ) { | |||
| connectionResultSubject.send(true) | |||
There was a problem hiding this comment.
아 얘가 보내는 값은 별로 중요하지 않은 건가요 ?? 항상 true를 보내고 UseCase에서 분기처리 ? ? ?
There was a problem hiding this comment.
뭔가 false를 보내줄 일이 있지 않을까.. 해서 남겨놓았습니다만..
조이가 보신대로 true를 보내주는 코드만 존재합니다.
변경하는게 맞을까요?
일단 true를 보내는 이유는 연결에 성공했기 때문입니다~!
| @@ -13,6 +13,9 @@ public final class WhiteboardUseCase: WhiteboardUseCaseInterface { | |||
| private var profileRepository: ProfileRepositoryInterface | |||
| private let whiteboardListSubject: CurrentValueSubject<[Whiteboard], Never> | |||
| public let whiteboardListPublisher: AnyPublisher<[Whiteboard], Never> | |||
| public let whiteboardConnectionSubject: PassthroughSubject<Bool, Never> | |||
eemdeeks
reviewed
Dec 3, 2024
| whiteboardRepository.connectionResultPublisher | ||
| .timeout(.seconds(3), scheduler: DispatchQueue.main) | ||
| .first() | ||
| .replaceEmpty(with: false) |
There was a problem hiding this comment.
timeout: 3초동안만 이벤트를 수신합니다first: 첫 번째 응답을 받고 구독을 취소합니다. (PR 내용의 일회성 구독을 위해 사용하였습니다)replaceEmpty: 응답을 못받았다면false를 방출합니다
종합적으로 3초동안 이벤트를 받는데 다른 참가자가 연결된 경우에도 이벤트를 받지 않기 위해 일회성 구독을 하고, 아무 값도 받아오지 못한 경우 실패를 위해 빈 값을fasle로 치환했습니다!!
|
eemdeeks
approved these changes
Dec 3, 2024
졌습니다.. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🌁 Background
📱 Screenshot
2024-12-03.3.49.24.mov
👩💻 Contents
WhiteboardRepository는 연결 성공 여부를Bool값으로 방출합니다.WhiteboardUseCase는 Join 요청 시 일회성 구독을 시작합니다. (다른 참여자가 참가했을 때 값을 받지 않기 위해서)WhiteboardViewController로 진입합니다.✅ Testing
유령 화이트 보드를 만드는 방법은 방을 만들고 참여자 입장 후 호스트가 먼저 나가고 참여자가 나가는 것입니다.
시뮬레이터에선 방이 사라지지만 실기기에서는 사라지지않습니다.
위에있는 동작 영상 속 첫 번째 셀은 유령 화이트보드로, 연결 시도시 Alert를 띄웁니다.
두번째 셀은 정상적인 방으로 3초 이내에 연결 요청이 수락되어 입장합니다.
📝 Review Note
disconnect처리하고있습니다.📣 Related Issue