-
Notifications
You must be signed in to change notification settings - Fork 0
Feature: 다시보기 완성 #299
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
Feature: 다시보기 완성 #299
Conversation
서버에서 받아온 데이터로 다시보기를 확인 할 수 있습니다.
✅ Deploy Preview for boarlog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생 많으셨습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM입니다! 너무 고생 많으셨어요 주완님! :)
console.log(result.data.logs); | ||
loadedDataRef.current = result.data.logs; | ||
scriptListRef.current = result.data.subtitles; | ||
localAudioRef.current!.src = result.data.audio_file; | ||
localAudioRef.current!.addEventListener("loadedmetadata", () => { | ||
totalTimeRef.current = localAudioRef.current!.duration * 1000; | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지금은 크게 중요한건 아니지만... 크롱님이 남겨주셨던 코드리뷰가 떠오르네요.. ㅎㅎㅎ 구조분해할당으로 한 번만 객체의 요소에 접근하도록 하는 것도 하나의 방법일 것 같습니다. 나중에 같이 개선해봐요 :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 좋아요!
서버에서 받아온 데이터로 다시보기를 확인 할 수 있습니다.
작업 개요
다시보기 데이터를 서버에서 받아서 실행 할 수 있습니다.
작업 사항
스크린샷
55.mp4