From 004e8d5fe9d78cc47ca79573f761e42d27a7511e Mon Sep 17 00:00:00 2001
From: Belen Curcio <curciobelen@gmail.com>
Date: Fri, 31 Mar 2017 12:37:11 -0300
Subject: [PATCH 1/4] Using wrapResponse

---
 graph/helpers/response.js                     | 31 +++++++++++++++++++
 graph/resolvers/root_mutation.js              | 29 +----------------
 .../client/components/Icon.js                 |  2 +-
 plugins/coral-plugin-respect/index.js         |  3 +-
 4 files changed, 35 insertions(+), 30 deletions(-)
 create mode 100644 graph/helpers/response.js

diff --git a/graph/helpers/response.js b/graph/helpers/response.js
new file mode 100644
index 000000000..ebdbc02ec
--- /dev/null
+++ b/graph/helpers/response.js
@@ -0,0 +1,31 @@
+const errors = require('../../errors');
+const {Error: {ValidationError}} = require('mongoose');
+
+/**
+ * Wraps up a promise to return an object with the resolution of the promise
+ * keyed at `key` or an error caught at `errors`.
+ */
+
+const wrapResponse = (key) => (promise) => {
+  return promise.then((value) => {
+    let res = {};
+    if (key) {
+      res[key] = value;
+    }
+    return res;
+  }).catch((err) => {
+    if (err instanceof errors.APIError) {
+      return {
+        errors: [err]
+      };
+    } else if (err instanceof ValidationError) {
+
+      // TODO: wrap this with one of our internal errors.
+      throw err;
+    }
+
+    throw err;
+  });
+};
+
+module.exports = wrapResponse;
diff --git a/graph/resolvers/root_mutation.js b/graph/resolvers/root_mutation.js
index a474407d2..3d5ea9ad9 100644
--- a/graph/resolvers/root_mutation.js
+++ b/graph/resolvers/root_mutation.js
@@ -1,33 +1,6 @@
-const {Error: {ValidationError}} = require('mongoose');
-const errors = require('../../errors');
+const wrapResponse = require('../helpers/response');
 const CommentsService = require('../../services/comments');
 
-/**
- * Wraps up a promise to return an object with the resolution of the promise
- * keyed at `key` or an error caught at `errors`.
- */
-const wrapResponse = (key) => (promise) => {
-  return promise.then((value) => {
-    let res = {};
-    if (key) {
-      res[key] = value;
-    }
-    return res;
-  }).catch((err) => {
-    if (err instanceof errors.APIError) {
-      return {
-        errors: [err]
-      };
-    } else if (err instanceof ValidationError) {
-
-      // TODO: wrap this with one of our internal errors.
-      throw err;
-    }
-
-    throw err;
-  });
-};
-
 const RootMutation = {
   createComment(_, {asset_id, parent_id, body}, {mutators: {Comment}}) {
     return wrapResponse('comment')(Comment.create({asset_id, parent_id, body}));
diff --git a/plugins/coral-plugin-respect/client/components/Icon.js b/plugins/coral-plugin-respect/client/components/Icon.js
index 8a39218dc..1e4dd30b9 100644
--- a/plugins/coral-plugin-respect/client/components/Icon.js
+++ b/plugins/coral-plugin-respect/client/components/Icon.js
@@ -1,5 +1,5 @@
 import React from 'react';
 
 export default () => (
-  <i className="fa fa-handshake-o" aria-hidden="true"></i>
+  <i className="fa fa-handshake-o" aria-hidden="true"/>
 );
diff --git a/plugins/coral-plugin-respect/index.js b/plugins/coral-plugin-respect/index.js
index 140a19ddf..935c2fad3 100644
--- a/plugins/coral-plugin-respect/index.js
+++ b/plugins/coral-plugin-respect/index.js
@@ -1,11 +1,12 @@
 const typeDefs = require('./server/typeDefs');
+const wrapResponse = require('../../graph/helpers/response');
 
 module.exports = {
   typeDefs,
   resolvers: {
     RootMutation: {
       createRespect(_, {respect: {item_id, item_type}}, {mutators: {Action}}) {
-        return Action.create({item_id, item_type, action_type: 'RESPECT'});
+        return wrapResponse(Action.create({item_id, item_type, action_type: 'RESPECT'}));
       }
     }
   },

From 1da5c570bf0f13c8f93390f8f16888322851f940 Mon Sep 17 00:00:00 2001
From: Wyatt Johnson <wyattjoh@gmail.com>
Date: Fri, 31 Mar 2017 09:41:18 -0600
Subject: [PATCH 2/4] Fixed syntax error

---
 plugins/coral-plugin-respect/index.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/coral-plugin-respect/index.js b/plugins/coral-plugin-respect/index.js
index 935c2fad3..2f94e208e 100644
--- a/plugins/coral-plugin-respect/index.js
+++ b/plugins/coral-plugin-respect/index.js
@@ -6,7 +6,7 @@ module.exports = {
   resolvers: {
     RootMutation: {
       createRespect(_, {respect: {item_id, item_type}}, {mutators: {Action}}) {
-        return wrapResponse(Action.create({item_id, item_type, action_type: 'RESPECT'}));
+        return wrapResponse('respect')(Action.create({item_id, item_type, action_type: 'RESPECT'}));
       }
     }
   },

From 0a77fe92bd7a040abb9c733c2c6e0f72f82abfa0 Mon Sep 17 00:00:00 2001
From: Belen Curcio <curciobelen@gmail.com>
Date: Fri, 31 Mar 2017 14:23:03 -0300
Subject: [PATCH 3/4] Adding translations and fully working respect button and
 styles

---
 plugins.json                                    |  2 +-
 plugins/coral-plugin-respect/client/config.json |  2 +-
 plugins/coral-plugin-respect/client/index.js    | 14 +++++++++++---
 plugins/coral-plugin-respect/client/style.css   |  7 +++++--
 4 files changed, 18 insertions(+), 7 deletions(-)

diff --git a/plugins.json b/plugins.json
index 6ac303c35..f135d8430 100644
--- a/plugins.json
+++ b/plugins.json
@@ -5,4 +5,4 @@
   "client": [
     "coral-plugin-respect"
   ]
-}
\ No newline at end of file
+}
diff --git a/plugins/coral-plugin-respect/client/config.json b/plugins/coral-plugin-respect/client/config.json
index 7d822b10f..ef9fed04f 100644
--- a/plugins/coral-plugin-respect/client/config.json
+++ b/plugins/coral-plugin-respect/client/config.json
@@ -1,4 +1,4 @@
 {
   "name": "Coral Plugin Respect",
   "slot": "Comment.Detail"
-}
\ No newline at end of file
+}
diff --git a/plugins/coral-plugin-respect/client/index.js b/plugins/coral-plugin-respect/client/index.js
index 341363571..de23f994b 100644
--- a/plugins/coral-plugin-respect/client/index.js
+++ b/plugins/coral-plugin-respect/client/index.js
@@ -2,6 +2,10 @@ import React from 'react';
 import styles from './style.css';
 import Icon from './components/Icon';
 
+import {I18n} from 'coral-framework';
+import translations from './translations.json';
+const lang = new I18n(translations);
+
 import {getActionSummary} from 'coral-framework/utils';
 
 class RespectButton extends React.Component {
@@ -15,7 +19,7 @@ class RespectButton extends React.Component {
   }
 
   handleClick = () => {
-    const {comment, postRespect, showSignInDialog, currentUser} = this.props.context;
+    const {comment, postRespect, showSignInDialog, currentUser, deleteAction} = this.props.context;
     const {localPost, localDelete} = this.state;
     const respect = getActionSummary('RespectActionSummary', comment);
     const respected = (respect && respect.current_user && !localDelete) || localPost;
@@ -54,6 +58,7 @@ class RespectButton extends React.Component {
 
     } else {
       this.setState((prev) => prev.localPost ? {...prev, localPost: null} : {...prev, localDelete: true});
+      deleteAction(localPost || respect.current_user.id);
     }
   }
 
@@ -61,6 +66,7 @@ class RespectButton extends React.Component {
     const {comment} = this.props.context;
     const {localPost, localDelete} = this.state;
     const respect = getActionSummary('RespectActionSummary', comment);
+    const respected = (respect && respect.current_user && !localDelete) || localPost;
     let count = respect ? respect.count : 0;
 
     if (localPost) {count += 1;}
@@ -68,8 +74,10 @@ class RespectButton extends React.Component {
 
     return (
       <div className={styles.Respect}>
-        <button onClick={this.handleClick}>
-          Respect
+        <button
+          className={respected ? styles.respected : ''}
+          onClick={this.handleClick} >
+          <span>{lang.t(respected ? 'respected' : 'respect')}</span>
           <Icon />
           {count > 0 && count}
         </button>
diff --git a/plugins/coral-plugin-respect/client/style.css b/plugins/coral-plugin-respect/client/style.css
index 7933f8d8e..485315bfa 100644
--- a/plugins/coral-plugin-respect/client/style.css
+++ b/plugins/coral-plugin-respect/client/style.css
@@ -14,11 +14,14 @@
          }
     }
 
-    .clicked {
+    .respected {
         color: #ffcc00;
+        i {
+          color: #161616;
+        }
     }
 
     i {
         padding: 0 5px;
     }
-}
\ No newline at end of file
+}

From c4decbd844b8d31da86f702599fa96915de071b0 Mon Sep 17 00:00:00 2001
From: Belen Curcio <curciobelen@gmail.com>
Date: Fri, 31 Mar 2017 14:31:44 -0300
Subject: [PATCH 4/4] Adding eslinrc

---
 .eslintrc.json                                | 15 ++----------
 .../client/.eslintrc.json                     | 23 +++++++++++++++++++
 2 files changed, 25 insertions(+), 13 deletions(-)
 create mode 100644 plugins/coral-plugin-respect/client/.eslintrc.json

diff --git a/.eslintrc.json b/.eslintrc.json
index 285f5f7dc..293657679 100644
--- a/.eslintrc.json
+++ b/.eslintrc.json
@@ -4,18 +4,9 @@
         "node": true
     },
     "extends": "eslint:recommended",
-    "parser": "babel-eslint",
     "parserOptions": {
-        "ecmaVersion": 2017,
-        "sourceType": "module",
-        "ecmaFeatures": {
-            "jsx": true,
-            "experimentalObjectRestSpread": true
-        }
+        "ecmaVersion": 2017
     },
-    "plugins": [
-        "react"
-    ],
     "rules": {
         "indent": ["error",
             2
@@ -62,8 +53,6 @@
         }],
         "newline-per-chained-call": ["error", {
             "ignoreChainWithDepth": 2
-        }],
-        "react/jsx-uses-react": "error",
-        "react/jsx-uses-vars": "error"
+        }]
     }
 }
diff --git a/plugins/coral-plugin-respect/client/.eslintrc.json b/plugins/coral-plugin-respect/client/.eslintrc.json
new file mode 100644
index 000000000..9fe56bd14
--- /dev/null
+++ b/plugins/coral-plugin-respect/client/.eslintrc.json
@@ -0,0 +1,23 @@
+{
+  "env": {
+    "browser": true,
+    "es6": true,
+    "mocha": true
+  },
+  "parserOptions": {
+    "sourceType": "module",
+    "ecmaFeatures": {
+      "experimentalObjectRestSpread": true,
+      "jsx": true
+    }
+  },
+  "parser": "babel-eslint",
+  "plugins": [
+    "react"
+  ],
+  "rules": {
+    "react/jsx-uses-react": "error",
+    "react/jsx-uses-vars": "error",
+    "no-console": ["warn", { "allow": ["warn", "error"] }]
+  }
+}