Skip to content
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

feat: 영상 재추출시 업로드 에러, 지도 인코딩중 표시 및 권한 처리 #331

Merged
merged 11 commits into from
Dec 14, 2023

Conversation

anyukyung
Copy link
Member

🧑‍🚀 PR 요약

해당 pr에서 작업한 내역을 적어주세요.

  • 영상 재추출시 업로드가 되지 않던 에러를 픽스했습니다..ㅎ
  • 영상 export에 전 exporter의 AVFileType을 적절하게 지정해주어야하는데, 현재 알 수 있는 정보인 pathExtension으로는 AVFileType을 알 수가 없었습니다..
  • 아래가 각 파일 형식 별 AVFileType의 rawValue인데요 그냥 url 전달하면 AVFileType을 리턴하는 extension 만들어서 사용했습니다
mov: com.apple.quicktime-movie
mp4: public.mpeg-4
m4v: public.mpeg-4-video
...

  • 지도에 인코딩중인 영상을 표시했구요 -> 이건 인환님이 spinner를 이용해서 잘 만들어두셨길래 스리슬쩍..
  • 또 홈과 지도에서 upload 버튼을 눌렀을 때 권한 처리를 했습니다. 이부분이 로직이 거의 똑같아서 코드상 겹치는게 많습니다 → 중복 코드를 줄일 수 있을 방법이 없을까 하다가 당장은 묘수가 떠오르지 않아 우선 동일하게 처리해두었습니당

스크린샷 2023-12-14 오후 2 19 13

📌 변경 사항

변경사항 및 주의 사항 (모듈 설치 등)을 적어주세요.

📸 ScreenShot

작동, 구현화면

RPReplay_Final1702531335.MP4

Linked Issue

close #300

Comment on lines 12 to 22
static func from(_ url: URL) -> AVFileType? {
let pathExtension = url.pathExtension
switch pathExtension {
case "mp4":
return .mp4
case "mov":
return .mov
default:
return nil
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

다른 파일 형식...도 괜찮겠지요?

Copy link
Member Author

Choose a reason for hiding this comment

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

이거 exporter supportFileType 한번 확인해보고 추가해보겠습니다 !!

Copy link
Collaborator

@loinsir loinsir left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~ 컨플릭트만 해결해주세용~

Copy link
Collaborator

@chopmozzi chopmozzi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@anyukyung anyukyung linked an issue Dec 14, 2023 that may be closed by this pull request
1 task
@anyukyung anyukyung merged commit 7ab724d into iOS/dev Dec 14, 2023
1 check passed
@anyukyung anyukyung deleted the iOS/feat#300 branch January 10, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 지도에서 업로드 중인 게시글 처리
3 participants