Merge pull request #2208 from mattermost/client-upgrades

Updating some client deps + eslint

1 files changed, 16 insertions, 2 deletions

diff --git a/web/react/components/view_image.jsx b/web/react/components/view_image.jsx
index 90885e495..713fbf80b 100644
--- a/web/react/components/view_image.jsx
+++ b/web/react/components/view_image.jsx
@@ -203,7 +203,9 @@ class ViewImageModal extends React.Component {
                     window.open(serverData.public_link);
                 }
             },
-            () => {}
+            () => {
+                //Do Nothing on error
+            }
         );
     }
 
@@ -383,6 +385,11 @@ function LoadingImagePreview({progress, loading}) {
     );
 }
 
+LoadingImagePreview.propTypes = {
+    progress: React.PropTypes.number,
+    loading: React.PropTypes.string
+};
+
 function ImagePreview({filename, fileUrl, fileInfo, maxHeight}) {
     let previewUrl;
     if (fileInfo.has_preview_image) {
@@ -405,4 +412,11 @@ function ImagePreview({filename, fileUrl, fileInfo, maxHeight}) {
     );
 }
 
-export default injectIntl(ViewImageModal);
\ No newline at end of file
+ImagePreview.propTypes = {
+    filename: React.PropTypes.string.isRequired,
+    fileUrl: React.PropTypes.string.isRequired,
+    fileInfo: React.PropTypes.object.isRequired,
+    maxHeight: React.PropTypes.number.isRequired
+};
+
+export default injectIntl(ViewImageModal);