From 07e9114e65046d791fc4f6ed90d6e2e938ad559d Mon Sep 17 00:00:00 2001
From: Daniel Gasienica <daniel@gasienica.ch>
Date: Tue, 13 Feb 2018 11:37:31 -0500
Subject: [PATCH] Document incoming and outgoing attachments fields

Note how outgoing message attachments only have 4 fields. This presumably means
the others are not used in our code and could be discarded for simplicity.
---
 js/modules/types/attachment.js | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/js/modules/types/attachment.js b/js/modules/types/attachment.js
index 8e65b20e79..a40e659de1 100644
--- a/js/modules/types/attachment.js
+++ b/js/modules/types/attachment.js
@@ -10,17 +10,25 @@ const { autoOrientImage } = require('../auto_orient_image');
 // 1 --> 2, etc., similar to how we do database migrations:
 const CURRENT_PROCESS_VERSION = 1;
 
-// // Fields
+// // Incoming message attachment fields
 // {
 //   id: string
 //   contentType: MIMEType
-//   key: ArrayBuffer
-//   size: integer
-//   thumbnail: ArrayBuffer
+//   data: ArrayBuffer
 //   digest: ArrayBuffer
 //   fileName: string
 //   flags: null
+//   key: ArrayBuffer
+//   size: integer
+//   thumbnail: ArrayBuffer
+// }
+
+// // Outgoing message attachment fields
+// {
+//   contentType: MIMEType
 //   data: ArrayBuffer
+//   fileName: string
+//   size: integer
 // }
 
 // Middleware