From a4417d78cd5f4750eebd47515ae7f18e89256432 Mon Sep 17 00:00:00 2001 From: Almeida Date: Thu, 2 Nov 2023 01:39:36 +0000 Subject: [PATCH] fix: remove options --- packages/discord.js/src/structures/User.js | 4 ++-- packages/discord.js/typings/index.d.ts | 2 +- packages/rest/__tests__/CDN.test.ts | 4 ++-- packages/rest/src/lib/CDN.ts | 14 +++++++------- 4 files changed, 12 insertions(+), 12 deletions(-) diff --git a/packages/discord.js/src/structures/User.js b/packages/discord.js/src/structures/User.js index fa2950062b47..a66f6eb10ff5 100644 --- a/packages/discord.js/src/structures/User.js +++ b/packages/discord.js/src/structures/User.js @@ -192,12 +192,12 @@ class User extends Base { /** * A link to the user's avatar decoration. - * @param {ImageURLOptions} [options={}] Options for the image URL + * @param {BaseImageURLOptions} [options={}] Options for the image URL * @returns {?string} */ avatarDecorationURL(options = {}) { if (this.avatarDecorationData) { - return this.client.rest.cdn.avatarDecoration(this.avatarDecorationData.asset, options); + return this.client.rest.cdn.avatarDecoration(this.avatarDecorationData.asset); } return this.avatarDecoration && this.client.rest.cdn.avatarDecoration(this.id, this.avatarDecoration, options); diff --git a/packages/discord.js/typings/index.d.ts b/packages/discord.js/typings/index.d.ts index 73f0a8228373..1f4954a5293f 100644 --- a/packages/discord.js/typings/index.d.ts +++ b/packages/discord.js/typings/index.d.ts @@ -3162,7 +3162,7 @@ export class User extends PartialTextBasedChannel(Base) { public get tag(): string; public username: string; public avatarURL(options?: ImageURLOptions): string | null; - public avatarDecorationURL(options?: ImageURLOptions): string | null; + public avatarDecorationURL(options?: BaseImageURLOptions): string | null; public bannerURL(options?: ImageURLOptions): string | null | undefined; public createDM(force?: boolean): Promise; public deleteDM(): Promise; diff --git a/packages/rest/__tests__/CDN.test.ts b/packages/rest/__tests__/CDN.test.ts index a06c041999ff..ed2600f4f614 100644 --- a/packages/rest/__tests__/CDN.test.ts +++ b/packages/rest/__tests__/CDN.test.ts @@ -33,8 +33,8 @@ test('avatar decoration default', () => { expect(cdn.avatarDecoration(id, hash)).toEqual(`${base}/avatar-decorations/${id}/${hash}.webp`); }); -test('avatar decoration presets default', () => { - expect(cdn.avatarDecoration(hash)).toEqual(`${base}/avatar-decoration-presets/${hash}.webp`); +test('avatar decoration preset', () => { + expect(cdn.avatarDecoration(hash)).toEqual(`${base}/avatar-decoration-presets/${hash}.png`); }); test('banner default', () => { diff --git a/packages/rest/src/lib/CDN.ts b/packages/rest/src/lib/CDN.ts index dd21c96819ad..7d2ac6540c1e 100644 --- a/packages/rest/src/lib/CDN.ts +++ b/packages/rest/src/lib/CDN.ts @@ -101,14 +101,13 @@ export class CDN { * Generates a user avatar decoration preset URL. * * @param asset - The avatar decoration hash - * @param options - Optional options for the avatar decoration preset */ - public avatarDecoration(asset: string, options?: Readonly): string; + public avatarDecoration(asset: string): string; /** * Generates a user avatar decoration URL. * - * @deprecated This overload is deprecated. Pass an hash instead. + * @deprecated This overload is deprecated. Pass a hash instead. * @param userId - The id of the user * @param userAvatarDecoration - The hash provided by Discord for this avatar decoration * @param options - Optional options for the avatar decoration @@ -116,19 +115,20 @@ export class CDN { public avatarDecoration( userId: string, userAvatarDecoration: string, + // eslint-disable-next-line @typescript-eslint/unified-signatures options?: Readonly, ): string; public avatarDecoration( userIdOrAsset: string, - assetOrOptions?: Readonly | string, + userAvatarDecoration?: string, options?: Readonly, ): string { - if (typeof assetOrOptions === 'string') { - return this.makeURL(`/avatar-decorations/${userIdOrAsset}/${assetOrOptions}`, options); + if (userAvatarDecoration) { + return this.makeURL(`/avatar-decorations/${userIdOrAsset}/${userAvatarDecoration}`, options); } - return this.dynamicMakeURL(`/avatar-decoration-presets/${userIdOrAsset}`, userIdOrAsset, assetOrOptions); + return this.makeURL(`/avatar-decoration-presets/${userIdOrAsset}`, { extension: 'png' }); } /**