Conversation
Sonny-Kor
requested review from
INYEKIM,
around-forest,
hyunjuntyler and
moral-life
as code owners
1 task
* 원인은 제출 시 StopPlayer()를 호출해주고있었음 , + 중복적으로 네비게이션 컨트롤러에서 stopPlayer()와 stopEngine()을 비동기로 처리하고있었음. * 중복적으로 동작하다보니 Publish해야하는 State가 꼬이게 됨. * ViewController 내부에서 제출 버튼을 누르면 노래가 멈추도록 수정
* viewType이 nil일 때, ingame 노래가 나오는 걸 확인
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
필수 리뷰어
@hyunjuntyler
관련 이슈
완료 및 수정 내역
테스트 방법 (선택)
리뷰 노트
해당 메서드는 GameAudioHelper.shared.stopPlaying() 메서드를 비동기로 호출 하고있는 메서드입니다.
Engine이 생기고 나서 부터 제출버튼을 눌러도 Engine은 멈추지 않고 재생이 되고 있었고, Game Navigation Controller 에서 뷰 전환 시 BGM을 재생해야 하기 때문에
stopPlaying()과stopEngine()을 호출하고 있어, Result에서 Publish 값들이 꼬인 것 같습니다.해결 방법으로 GameNavigationController 에서 노래 정지 부분을 삭제하고, SubmitMusic, Answer, Humming, Rehumming ViewController 에서 제출 버튼 을 누를 때 노래 정지되도록 수정하였습니다.
추가적으로 Navigaiton Push로 뷰를 이동하다 보니,
Audio Player Component들의 deinit이 호출되지 않아, Binding을 계속 유지하고 있는 문제가 있어, ViewController에서 따로 unbind() 를 호출하도록 수정하였습니다.
스크린샷 (선택)