chore: 각종 문제점 개선 #293
chore: 각종 문제점 개선 #293
Conversation
| @@ -36,7 +36,7 @@ final class MockReportWorker: ReportWorkerProtocol { | |||
| do { | |||
| let bodyParameters = ReportDTO( | |||
| memberId: nil, | |||
| boardID: 1, | |||
| boardId: 1, | |||
There was a problem hiding this comment.
엇 이건 원래대로 ID가 더 적절한 것 같습니다.
There was a problem hiding this comment.
서버에서 response주는게 저거라 ID로 해서 오류가 난 거였습니다.
| private func transPostToVideo(_ posts: [Post]) async -> [Models.PlaybackVideo] { | ||
| return await posts.asyncCompactMap { post in | ||
| private func transPostToVideo(_ posts: [Post]) -> [Models.PlaybackVideo] { | ||
| return posts.compactMap { post in |
There was a problem hiding this comment.
이제 여기서 이미지 로드를 기다리지 않아도 되어서 확실히 성능 향상이 있을 것 같네요
고생하셨습니다...
| self.interactor?.loadProfileImageAndLocation(with: Models.LoadProfileImageAndLocation.Request( | ||
| curCell: cell, | ||
| profileImageURL: playbackVideo.displayedPost.member.profileImageURL, | ||
| latitude: playbackVideo.displayedPost.board.latitude, | ||
| longitude: playbackVideo.displayedPost.board.longitude)) |
There was a problem hiding this comment.
오 확실히 이렇게 필요한 부분에서만 호출되어서 로드되면 모든 데이터를 기다리지 않게 되겠네요
| @@ -60,7 +60,7 @@ final class ProfileInteractor: ProfileBusinessLogic, ProfileDataStore { | |||
| guard let userProfile = await userWorker?.fetchProfile(by: profileId) else { | |||
| return false | |||
| } | |||
|
|
|||
| posts = [] | |||
There was a problem hiding this comment.
놓친 부분 찾아주셔서 감사합니다.
태그 재생목록에도 같은 실수를 했는데, 이건 제가 지금 테스트 코드 짜면서 수정할게요!
|
@loinsir 네이밍 재고가 boardID말씀하신건가용?? |
| @@ -10,6 +10,6 @@ import Foundation | |||
|
|
|||
| struct ReportDTO: Codable { | |||
| let memberId: Int? | |||
| let boardID: Int | |||
| let boardId: Int | |||
There was a problem hiding this comment.
CodingKeys를 쓰면 이 id와 ID의 혼란이 조금은 나아질지도..
There was a problem hiding this comment.
좋은 생각입니다. 반영할게요.
| async let profileImageData = self.worker?.fetchImageData(with: request.profileImageURL) | ||
| async let location: String? = self.worker?.transLocation(latitude: request.latitude, longitude: request.longitude) |
| @@ -51,6 +51,7 @@ final class PlaybackView: UIView { | |||
| button.layer.borderColor = UIColor.layoverWhite.cgColor | |||
| button.backgroundColor = .layoverWhite | |||
| button.clipsToBounds = true | |||
| button.imageView?.contentMode = .scaleAspectFill | |||
| return button | |||
| }() | |||
|
|
|||
There was a problem hiding this comment.
그.. 이 이슈랑 관련된건 아니긴한데 playbackView 내부 Profile 버튼의 borderColor를
grey400으로 바까주실 수 있나여 ㅎ
There was a problem hiding this comment.
UIButton이 아니라 Button의 이미지의 border컬러가 맞을까요?
resetCode 추가
| enum CodingKeys: String, CodingKey { | ||
| case memberID = "memberId" | ||
| case boardID = "boardId" | ||
| case reportType | ||
| } |
네 맞습니다 |
🧑🚀 PR 요약
📌 변경 사항
ReportDTO 변경했습니다.
cell은 영상 먼저 로드하고 프로필 이미지와 로케이션을 로드합니다. 딜레이 부분을 좀 해결한 것 같습니다.
Linked Issue
close #289