feat: 동영상 선택, 편집 기능 #152
Merged
feat: 동영상 선택, 편집 기능 #152
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
chopmozzi
reviewed
Nov 30, 2023
loinsir
reviewed
Nov 30, 2023
loinsir
reviewed
Nov 30, 2023
loinsir
reviewed
Nov 30, 2023
Comment on lines
+106
to
+109
| @objc private func soundButtonDidTap() { | ||
| loopingPlayerView.player?.isMuted.toggle() | ||
| soundButton.isSelected.toggle() | ||
| } |
There was a problem hiding this comment.
헉 이거 동영상에서 소리 지워주는 기능 아니었나요??!?!
There was a problem hiding this comment.
비디오에서 소리를 제거하는 작업이 파일 자체를 건드려야하는건데, 버튼으로는 계속 껐다켰다할수있잖아요
껐다켰다할때마다, 혹은 업로드 화면으로 넘길때마다 파일 다시만들어서 asset에서 videotrack만 뽑는 작업 하는건 너무 비효율적이니까,
업로드 화면으로 넘어갈때 mute 유무 넘겨주고, 업로드 버튼 누를때 저 작업 해주는게 나을 것 같습니다!
이러한 이유로 audio 제거는 업로드 pr에서 구현하겠습니다ㅏ 😊
loinsir
reviewed
Nov 30, 2023
loinsir
reviewed
Nov 30, 2023
loinsir
reviewed
Nov 30, 2023
loinsir
reviewed
Nov 30, 2023
loinsir
reviewed
Nov 30, 2023
loinsir
reviewed
Nov 30, 2023
loinsir
reviewed
Nov 30, 2023
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.
🧑🚀 PR 요약
작업량이 많진 않은데, 처음 써보는 것들이라 학습하는데 시간이 좀 걸렸습니다!
PHPicker를 이용한 동영상 선택기능
PHPicker를 이용해서 동영상을 가져오기위해loadFileRepresentation사용했습니다.Note
이 메소드는
컴플리션 핸들러가 반환되기 전!까지만 선택된 동영상의 임시파일 URL을 유지시킵니다.그래서 핸들러가 반환되면 임시파일은 즉시 삭제됩니다..
VideoFileWorker로 분리했습니다전달받은 URL로 영상편집
UIViewEditorViewController를 이용해 영상을 편집합니다.두배가돼..메모리가 두두두.. 두배가돼..
3초~60초 영상 길이 제한
loadFileRepresentation메소드는 컴플리션 핸들러가 끝나면 영상이 사라지고, duration을 가져올 수 있는 load 메소드는 비동기였기 때문이죠...UIViewEditorViewController에서 맥시멈 길이만 제한할 수 있어서 어차피 소용이 없더군요.. 그래서 우선 영상 길이에 따라 다음 버튼을 활성화, 비활성화되게 해두었습니다📌 변경 사항
📸 ScreenShot
RPReplay_Final1701333770.MP4
Linked Issue
close #115