Skip to content

Commit e5c35c6

Browse files
author
Rajat
committed
Coded recommended changes
1 parent 46e23c0 commit e5c35c6

File tree

4 files changed

+16
-11
lines changed

4 files changed

+16
-11
lines changed

apps/api/src/media-settings/service.ts

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,31 @@
11
import * as queries from "./queries";
22
import { MediaSettingsResponse } from "./schemas";
3+
import {
4+
thumbnailHeight as defaultThumbnailHeight,
5+
thumbnailWidth as defaultThumbnailWidth,
6+
} from "../config/constants";
37

48
export async function getMediaSettings(
59
userId: string,
610
apikey: string,
7-
): Promise<MediaSettingsResponse | null> {
11+
): Promise<MediaSettingsResponse> {
812
const mediaSettings = await queries.getMediaSettings(userId, apikey);
913

1014
if (!mediaSettings) {
11-
return null; // Return null if not found to match the type
15+
return {
16+
useWebP: false,
17+
webpOutputQuality: 0,
18+
thumbnailHeight: defaultThumbnailHeight,
19+
thumbnailWidth: defaultThumbnailWidth,
20+
};
1221
}
1322

1423
return {
1524
useWebP: mediaSettings.useWebP || false,
1625
webpOutputQuality: mediaSettings.webpOutputQuality || 0,
17-
thumbnailHeight: mediaSettings.thumbnailHeight || 0,
18-
thumbnailWidth: mediaSettings.thumbnailWidth || 0,
26+
thumbnailHeight:
27+
mediaSettings.thumbnailHeight || defaultThumbnailHeight,
28+
thumbnailWidth: mediaSettings.thumbnailWidth || defaultThumbnailWidth,
1929
};
2030
}
2131

apps/api/src/media/schemas.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export const mediaResponseSchema = Joi.object<MediaResponse>({
3333
size: Joi.number().required(),
3434
access: Joi.string().valid("public", "private").required(),
3535
file: Joi.string().uri().required(),
36-
thumbnail: Joi.string().uri().required(),
36+
thumbnail: Joi.string().uri().optional(),
3737
caption: Joi.string().optional().allow(""),
3838
group: Joi.string().optional(),
3939
});

apps/api/src/swagger-generator.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -147,10 +147,6 @@ swaggerAutogen()(outputFile, routes, doc).then(() => {
147147
});
148148
}
149149

150-
if (apiPath === "/media/create" && method === "post") {
151-
operation.security = [{ apiKeyAuth: [] }];
152-
}
153-
154150
if (apiPath === "/health" && method === "get") {
155151
operation.security = [];
156152
}

apps/api/src/swagger_output.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -857,8 +857,7 @@
857857
"mimeType",
858858
"size",
859859
"access",
860-
"file",
861-
"thumbnail"
860+
"file"
862861
],
863862
"additionalProperties": false
864863
},

0 commit comments

Comments
 (0)