feat(MessageEmbed): add missing proxyURL property to video (discordjs…

…#3109) * Added missing property to MessageEmbed.video * Updated typings for MessageEmbed.video
richardfuca · Feb 24, 2019 · 7006f00 · 7006f00
1 parent bc0a761
commit 7006f00
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/structures/MessageEmbed.js b/src/structures/MessageEmbed.js
@@ -100,11 +100,17 @@ class MessageEmbed {
      * The video of this embed (if there is one)
      * @type {?Object}
      * @property {string} url URL of this video
+     * @property {string} proxyURL ProxyURL for this video
      * @property {number} height Height of this video
      * @property {number} width Width of this video
      * @readonly
      */
-    this.video = data.video;
+    this.video = data.video ? {
+      url: data.video.url,
+      proxyURL: data.video.proxy_url,
+      height: data.video.height,
+      width: data.video.width,
+    } : null;
 
     /**
      * The author of this embed (if there is one)

diff --git a/typings/index.d.ts b/typings/index.d.ts
@@ -713,7 +713,7 @@ declare module 'discord.js' {
 		public title: string;
 		public type: string;
 		public url: string;
-		public readonly video: { url?: string; height?: number; width?: number };
+		public readonly video: { url?: string; proxyURL?: string; height?: number; width?: number };
 		public addBlankField(inline?: boolean): this;
 		public addField(name: StringResolvable, value: StringResolvable, inline?: boolean): this;
 		public attachFiles(file: (MessageAttachment | FileOptions | string)[]): this;