Skip to content

Conversation

@hsw1920
Copy link
Collaborator

@hsw1920 hsw1920 commented Dec 2, 2024

🤔 배경

편집화면에서 게스트가 SharePhotoView로 이동될 때는 bindNotification()을 통해 다음 화면으로 이동합니다.
이때 호스트가 Notification을 2번 중복으로 post하는 문제가 발생하였습니다.

📃 작업 내역

  • 호스트가 Notification을 2번 중복으로 post하는 문제 해결

✅ 리뷰 노트

  • 중복으로 post하는 코드를 하나 삭제하였습니다.
  • EditPhotoRoomHostViewController 파일을 확인해주세요.

🎨 스크린샷

X

🚀 테스트 방법

X

Notification post를 두 번 보내는 코드를 하나 삭제
@hsw1920 hsw1920 added the 🔧 fix 버그 수정 label Dec 2, 2024
@hsw1920 hsw1920 self-assigned this Dec 2, 2024
@hsw1920 hsw1920 linked an issue Dec 2, 2024 that may be closed by this pull request
@hsw1920 hsw1920 removed the request for review from 0Hooni December 2, 2024 11:36
Copy link
Member

@youn9k youn9k left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@Kiyoung-Kim-57 Kiyoung-Kim-57 left a comment

Choose a reason for hiding this comment

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

LGTVVM!

@hsw1920 hsw1920 merged commit 0a6303b into develop Dec 3, 2024
1 check passed
@hsw1920 hsw1920 deleted the fix/#170-next-noti-repair branch December 3, 2024 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔧 fix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

다음 화면으로 넘어가는 이벤트를 브로드캐스팅 한다

5 participants