Skip to content

Commit

Permalink
We don't need to load complete description anymore
Browse files Browse the repository at this point in the history
It's already included in classic payload
  • Loading branch information
Chocobozzz committed Aug 12, 2024
1 parent 26ca98f commit 1122e27
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 14 deletions.
4 changes: 1 addition & 3 deletions client/src/app/+videos/+video-edit/video-update.resolver.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,9 +37,7 @@ export class VideoUpdateResolver {

private buildVideoObservables (video: VideoDetails) {
return [
this.videoService
.loadCompleteDescription(video.descriptionPath)
.pipe(map(description => Object.assign(video, { description }))),
of(video),

this.videoService.getSource(video.id),

Expand Down
11 changes: 0 additions & 11 deletions client/src/app/shared/shared-main/video/video.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -379,17 +379,6 @@ export class VideoService {

// ---------------------------------------------------------------------------

loadCompleteDescription (descriptionPath: string) {
return this.authHttp
.get<{ description: string }>(environment.apiUrl + descriptionPath)
.pipe(
map(res => res.description),
catchError(err => this.restExtractor.handleError(err))
)
}

// ---------------------------------------------------------------------------

generateDownloadUrl (options: {
video: Video
files: VideoFile[]
Expand Down

0 comments on commit 1122e27

Please sign in to comment.