From 5650a669162ecbce01ba632cdf12db70e23e51f6 Mon Sep 17 00:00:00 2001
From: Patrick Fic <snaptsoft@gmail.com>
Date: Mon, 29 Jun 2020 09:58:54 -0700
Subject: [PATCH] Added remove from scoreboard functionality BOD-91

---
 bodyshop_translations.babel                   | 75 +++++++++++++++++-
 .../scoreboard-display.component.jsx          |  7 +-
 .../scoreboard-jobs-list.component.jsx        | 76 +++++++++++++++++++
 .../scorebard-remove-button.component.jsx     | 33 ++++++++
 client/src/graphql/scoreboard.queries.js      |  9 +++
 client/src/translations/en_us/common.json     |  7 ++
 client/src/translations/es/common.json        |  7 ++
 client/src/translations/fr/common.json        |  7 ++
 8 files changed, 217 insertions(+), 4 deletions(-)
 create mode 100644 client/src/components/scoreboard-jobs-list/scoreboard-jobs-list.component.jsx
 create mode 100644 client/src/components/scoreboard-remove-button/scorebard-remove-button.component.jsx

diff --git a/bodyshop_translations.babel b/bodyshop_translations.babel
index f4b6ffd48..dc2a4d597 100644
--- a/bodyshop_translations.babel
+++ b/bodyshop_translations.babel
@@ -1,4 +1,4 @@
-<babeledit_project version="1.2" be_version="2.6.1">
+<babeledit_project be_version="2.6.1" version="1.2">
     <!--
 
     BabelEdit project file
@@ -16209,6 +16209,79 @@
                     <folder_node>
                         <name>scoreboard</name>
                         <children>
+                            <folder_node>
+                                <name>errors</name>
+                                <children>
+                                    <concept_node>
+                                        <name>removing</name>
+                                        <definition_loaded>false</definition_loaded>
+                                        <description></description>
+                                        <comment></comment>
+                                        <default_text></default_text>
+                                        <translations>
+                                            <translation>
+                                                <language>en-US</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>es-MX</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>fr-CA</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                        </translations>
+                                    </concept_node>
+                                </children>
+                            </folder_node>
+                            <folder_node>
+                                <name>fields</name>
+                                <children>
+                                    <concept_node>
+                                        <name>bodyhrs</name>
+                                        <definition_loaded>false</definition_loaded>
+                                        <description></description>
+                                        <comment></comment>
+                                        <default_text></default_text>
+                                        <translations>
+                                            <translation>
+                                                <language>en-US</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>es-MX</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>fr-CA</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                        </translations>
+                                    </concept_node>
+                                    <concept_node>
+                                        <name>painthrs</name>
+                                        <definition_loaded>false</definition_loaded>
+                                        <description></description>
+                                        <comment></comment>
+                                        <default_text></default_text>
+                                        <translations>
+                                            <translation>
+                                                <language>en-US</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>es-MX</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>fr-CA</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                        </translations>
+                                    </concept_node>
+                                </children>
+                            </folder_node>
                             <folder_node>
                                 <name>labels</name>
                                 <children>
diff --git a/client/src/components/scoreboard-display/scoreboard-display.component.jsx b/client/src/components/scoreboard-display/scoreboard-display.component.jsx
index 759bfc2e5..f25f5a21c 100644
--- a/client/src/components/scoreboard-display/scoreboard-display.component.jsx
+++ b/client/src/components/scoreboard-display/scoreboard-display.component.jsx
@@ -1,8 +1,8 @@
 import React from "react";
-
-import ScoreboardTargetsTable from "../scoreboard-targets-table/scoreboard-targets-table.component";
-import ScoreboardLastDays from "../scoreboard-last-days/scoreboard-last-days.component";
 import ScoreboardChart from "../scoreboard-chart/scoreboard-chart.component";
+import ScoreboardJobsList from "../scoreboard-jobs-list/scoreboard-jobs-list.component";
+import ScoreboardLastDays from "../scoreboard-last-days/scoreboard-last-days.component";
+import ScoreboardTargetsTable from "../scoreboard-targets-table/scoreboard-targets-table.component";
 
 export default function ScoreboardDisplayComponent({ scoreboardSubscription }) {
   const { loading, error, data } = scoreboardSubscription;
@@ -24,6 +24,7 @@ export default function ScoreboardDisplayComponent({ scoreboardSubscription }) {
   return (
     <div>
       <ScoreboardTargetsTable />
+      <ScoreboardJobsList scoreBoardlist={scoreBoardlist} />
       <ScoreboardLastDays sbEntriesByDate={sbEntriesByDate} />
       <ScoreboardChart sbEntriesByDate={sbEntriesByDate} />
     </div>
diff --git a/client/src/components/scoreboard-jobs-list/scoreboard-jobs-list.component.jsx b/client/src/components/scoreboard-jobs-list/scoreboard-jobs-list.component.jsx
new file mode 100644
index 000000000..7a7813e40
--- /dev/null
+++ b/client/src/components/scoreboard-jobs-list/scoreboard-jobs-list.component.jsx
@@ -0,0 +1,76 @@
+import React from "react";
+import { Dropdown, Button, Table } from "antd";
+import { useTranslation } from "react-i18next";
+import { Link } from "react-router-dom";
+import ScoreboardRemoveButton from "../scoreboard-remove-button/scorebard-remove-button.component";
+import { DateFormatter } from "../../utils/DateFormatter";
+
+export default function ScoreboardJobsList({ scoreBoardlist }) {
+  const { t } = useTranslation();
+
+  const columns = [
+    {
+      title: t("jobs.fields.ro_number"),
+      dataIndex: "ro_number",
+      key: "ro_number",
+      render: (text, record) => (
+        <Link to={"/manage/jobs/" + record.job.id}>{record.job.ro_number}</Link>
+      ),
+    },
+    {
+      title: t("jobs.fields.est_number"),
+      dataIndex: "est_number",
+      key: "est_number",
+      render: (text, record) => (
+        <Link to={"/manage/jobs/" + record.job.id}>
+          {record.job.est_number}
+        </Link>
+      ),
+    },
+    {
+      title: t("scoreboard.fields.date"),
+      dataIndex: "date",
+      key: "date",
+      render: (text, record) => <DateFormatter>{record.date}</DateFormatter>,
+    },
+    {
+      title: t("scoreboard.fields.painthrs"),
+      dataIndex: "painthrs",
+      key: "painthrs",
+    },
+    {
+      title: t("scoreboard.fields.bodyhrs"),
+      dataIndex: "bodyhrs",
+      key: "bodyhrs",
+    },
+    {
+      title: t("general.labels.actions"),
+      dataIndex: "actions",
+      key: "actions",
+      render: (text, record) => (
+        <div>
+          <ScoreboardRemoveButton scoreboardId={record.id} />
+        </div>
+      ),
+    },
+  ];
+
+  const overlay = (
+    <div style={{ width: "30vw" }}>
+      <Table
+        size='small'
+        pagination={false}
+        columns={columns}
+        rowKey='id'
+        dataSource={scoreBoardlist}
+        scroll={{ x: true, y: "15rem" }}
+      />
+    </div>
+  );
+
+  return (
+    <Dropdown trigger={["click"]} overlay={overlay}>
+      <Button>Jobs</Button>
+    </Dropdown>
+  );
+}
diff --git a/client/src/components/scoreboard-remove-button/scorebard-remove-button.component.jsx b/client/src/components/scoreboard-remove-button/scorebard-remove-button.component.jsx
new file mode 100644
index 000000000..56454e561
--- /dev/null
+++ b/client/src/components/scoreboard-remove-button/scorebard-remove-button.component.jsx
@@ -0,0 +1,33 @@
+import React, { useState } from "react";
+import { useTranslation } from "react-i18next";
+import { Button, notification } from "antd";
+import { DeleteFilled } from "@ant-design/icons";
+import { useMutation } from "@apollo/react-hooks";
+import { DELETE_SCOREBOARD_ENTRY } from "../../graphql/scoreboard.queries";
+
+export default function ScoreboardRemoveButton({ scoreboardId }) {
+  const { t } = useTranslation();
+  const [deleteScoreboardEntry] = useMutation(DELETE_SCOREBOARD_ENTRY);
+  const [loading, setLoading] = useState(false);
+  const handleDelete = async (e) => {
+    e.stopPropagation();
+    setLoading(true);
+    const result = await deleteScoreboardEntry({
+      variables: { sbId: scoreboardId },
+    });
+
+    if (!!result.errors) {
+      notification["error"]({
+        message: t("scoreboard.errors.removing", {
+          message: JSON.stringify(result.errors),
+        }),
+      });
+    }
+    setLoading(false);
+  };
+  return (
+    <Button onClick={handleDelete} loading={loading}>
+      <DeleteFilled />
+    </Button>
+  );
+}
diff --git a/client/src/graphql/scoreboard.queries.js b/client/src/graphql/scoreboard.queries.js
index fd9c659e3..47254ec46 100644
--- a/client/src/graphql/scoreboard.queries.js
+++ b/client/src/graphql/scoreboard.queries.js
@@ -10,7 +10,16 @@ export const SUBSCRIPTION_SCOREBOARD = gql`
       job {
         id
         ro_number
+        est_number
       }
     }
   }
 `;
+
+export const DELETE_SCOREBOARD_ENTRY = gql`
+  mutation DELETE_SCOREBOARD_ENTRY($sbId: uuid!) {
+    delete_scoreboard_by_pk(id: $sbId) {
+      id
+    }
+  }
+`;
diff --git a/client/src/translations/en_us/common.json b/client/src/translations/en_us/common.json
index 6fda6319a..9d898d012 100644
--- a/client/src/translations/en_us/common.json
+++ b/client/src/translations/en_us/common.json
@@ -1015,6 +1015,13 @@
 			}
 		},
 		"scoreboard": {
+			"errors": {
+				"removing": "Error removing job from scoreboard. {{message}}"
+			},
+			"fields": {
+				"bodyhrs": "Body Hours",
+				"painthrs": "Paint Hours"
+			},
 			"labels": {
 				"asoftodaytarget": "As of Today",
 				"dailytarget": "Daily",
diff --git a/client/src/translations/es/common.json b/client/src/translations/es/common.json
index b4cddb936..e4f28d31a 100644
--- a/client/src/translations/es/common.json
+++ b/client/src/translations/es/common.json
@@ -1015,6 +1015,13 @@
 			}
 		},
 		"scoreboard": {
+			"errors": {
+				"removing": ""
+			},
+			"fields": {
+				"bodyhrs": "",
+				"painthrs": ""
+			},
 			"labels": {
 				"asoftodaytarget": "",
 				"dailytarget": "",
diff --git a/client/src/translations/fr/common.json b/client/src/translations/fr/common.json
index a6574ffec..201991f24 100644
--- a/client/src/translations/fr/common.json
+++ b/client/src/translations/fr/common.json
@@ -1015,6 +1015,13 @@
 			}
 		},
 		"scoreboard": {
+			"errors": {
+				"removing": ""
+			},
+			"fields": {
+				"bodyhrs": "",
+				"painthrs": ""
+			},
 			"labels": {
 				"asoftodaytarget": "",
 				"dailytarget": "",