From 488a8a52ea856ffd980cd44568dd298f898fc6f9 Mon Sep 17 00:00:00 2001
From: Faris Ansari <netchamp.faris@gmail.com>
Date: Thu, 2 Jan 2020 22:40:18 +0530
Subject: [PATCH] fix: GetStarted

- Mark cards as checked when task is completed
---
 models/doctype/GetStarted/GetStarted.js |  46 ++++-
 src/components/Icons/24/green-check.vue |  26 +++
 src/pages/GetStarted.vue                | 216 +++++++++++++++++++-----
 src/pages/SetupWizard/setupCompany.js   |   1 +
 4 files changed, 240 insertions(+), 49 deletions(-)
 create mode 100644 src/components/Icons/24/green-check.vue

diff --git a/models/doctype/GetStarted/GetStarted.js b/models/doctype/GetStarted/GetStarted.js
index 66d14a3c..05c9f313 100644
--- a/models/doctype/GetStarted/GetStarted.js
+++ b/models/doctype/GetStarted/GetStarted.js
@@ -3,14 +3,48 @@ module.exports = {
   isSingle: 1,
   fields: [
     {
-      fieldname: 'needOnboarding',
-      label: 'Need Onboarding',
-      fieldtype: 'Check',
-      default: 1
+      fieldname: 'onboardingComplete',
+      label: 'Onboarding Complete',
+      fieldtype: 'Check'
     },
     {
-      fieldname: 'salesItemCreated',
-      label: 'Sales Item Created',
+      fieldname: 'companySetup',
+      label: 'Company Setup',
+      fieldtype: 'Check'
+    },
+    {
+      fieldname: 'systemSetup',
+      label: 'System Setup',
+      fieldtype: 'Check'
+    },
+    {
+      fieldname: 'invoiceSetup',
+      label: 'Invoice Setup',
+      fieldtype: 'Check'
+    },
+    {
+      fieldname: 'itemCreated',
+      label: 'Item Created',
+      fieldtype: 'Check'
+    },
+    {
+      fieldname: 'customerCreated',
+      label: 'Customer Created',
+      fieldtype: 'Check'
+    },
+    {
+      fieldname: 'supplierCreated',
+      label: 'Supplier Created',
+      fieldtype: 'Check'
+    },
+    {
+      fieldname: 'invoiceCreated',
+      label: 'Invoice Created',
+      fieldtype: 'Check'
+    },
+    {
+      fieldname: 'billCreated',
+      label: 'Bill Created',
       fieldtype: 'Check'
     }
   ]
diff --git a/src/components/Icons/24/green-check.vue b/src/components/Icons/24/green-check.vue
new file mode 100644
index 00000000..2b30b685
--- /dev/null
+++ b/src/components/Icons/24/green-check.vue
@@ -0,0 +1,26 @@
+<template>
+  <svg width="24" height="24" xmlns="http://www.w3.org/2000/svg">
+    <g fill="none" fill-rule="evenodd">
+      <path fill="#FFF" d="M-589-585H611v907H-589z" />
+      <path
+        d="M12 0C5.373 0 0 5.373 0 12s5.373 12 12 12 12-5.373 12-12C23.98 5.38 18.62.02 12 0z"
+        fill="#92D336"
+        fill-rule="nonzero"
+      />
+      <path
+        stroke="#FFF"
+        stroke-width="1.5"
+        stroke-linecap="round"
+        stroke-linejoin="round"
+        d="M8 12.756l2.222 2.4L16 9.022"
+      />
+    </g>
+  </svg>
+</template>
+<script>
+import Base from '../base';
+export default {
+  name: 'IconGreenCheck',
+  extends: Base
+};
+</script>
diff --git a/src/pages/GetStarted.vue b/src/pages/GetStarted.vue
index 820c070b..8e086a5a 100644
--- a/src/pages/GetStarted.vue
+++ b/src/pages/GetStarted.vue
@@ -19,22 +19,44 @@
           >
             <div
               class="flex flex-col justify-between border rounded-lg p-6 h-full hover:shadow-md cursor-pointer"
-              @mouseenter="() => (item.showActions = true)"
-              @mouseleave="() => (item.showActions = false)"
+              @mouseenter="() => (activeCard = item.key)"
+              @mouseleave="() => (activeCard = null)"
             >
               <div>
                 <component
-                  v-show="!item.showActions"
-                  :is="getIconComponent(item.icon)"
+                  v-show="activeCard !== item.key && !isCompleted(item)"
+                  :is="getIconComponent(item)"
                   class="mb-4"
                 />
+                <Icon
+                  v-show="isCompleted(item)"
+                  name="green-check"
+                  size="24"
+                  class="w-5 h-5 mb-4"
+                />
                 <h3 class="font-medium">{{ item.label }}</h3>
                 <p class="mt-2 text-sm text-gray-800">
                   {{ item.description }}
                 </p>
               </div>
-              <div class="mt-2 flex" v-show="item.showActions">
-                <Button class="leading-tight" type="primary">
+              <div
+                class="mt-2 flex"
+                v-show="activeCard === item.key && !isCompleted(item)"
+              >
+                <Button
+                  class="leading-tight"
+                  type="primary"
+                  v-on="
+                    item.action
+                      ? {
+                          click: () => {
+                            item.action();
+                            activeCard = null;
+                          }
+                        }
+                      : null
+                  "
+                >
                   <span class="text-white text-base">
                     {{ _('Setup') }}
                   </span>
@@ -54,130 +76,238 @@
 </template>
 
 <script>
+import frappe from 'frappejs';
+import { _ } from 'frappejs/utils';
 import PageHeader from '@/components/PageHeader';
 import Icon from '@/components/Icon';
 import Button from '@/components/Button';
+import { openSettings } from '@/utils';
 
 export default {
   name: 'GetStarted',
   components: {
     PageHeader,
-    Button
+    Button,
+    Icon
   },
-  data() {
-    return {
-      sections: [
+  computed: {
+    sections() {
+      /* eslint-disable vue/no-side-effects-in-computed-properties */
+      return [
         {
-          label: 'Organisation',
+          label: _('Organisation'),
 
           items: [
             {
-              label: 'General',
+              key: 'General',
+              label: _('General'),
               icon: 'general',
               description:
                 'Setup your company information, email, country and fiscal year',
-              showActions: false
+              fieldname: 'companySetup'
             },
             {
-              label: 'System',
+              key: 'System',
+              label: _('System'),
               icon: 'general',
               description:
                 'Setup system defaults like date format and currency precision',
-              showActions: false
+              fieldname: 'systemSetup',
+              action() {
+                openSettings('System');
+              }
             },
             {
-              label: 'Invoice',
+              key: 'Invoice',
+              label: _('Invoice'),
               icon: 'invoice',
               description:
-                'Customize your invoices by adding a logo and company address',
-              showActions: false
+                'Customize your invoices by adding a logo and address details',
+              fieldname: 'invoiceSetup',
+              action() {
+                openSettings('Invoice');
+              }
             }
           ]
         },
         {
-          label: 'Accounts',
+          label: _('Accounts'),
 
           items: [
             {
-              label: 'Review Accounts',
+              key: 'Review Accounts',
+              label: _('Review Accounts'),
               icon: 'review-ac',
               description:
                 'Review your chart of accounts, add any account or tax heads as needed',
-              showActions: false
+              action: () => this.$router.push('/chart-of-accounts')
             },
             {
-              label: 'Opening Balances',
+              key: 'Opening Balances',
+              label: _('Opening Balances'),
               icon: 'opening-ac',
               description:
-                'Setup your opening balances before performing any accounting entries',
-              showActions: false
+                'Setup your opening balances before performing any accounting entries'
             },
             {
-              label: 'Add Taxes',
+              key: 'Add Taxes',
+              label: _('Add Taxes'),
               icon: 'percentage',
               description:
                 'Setup your tax templates for your sales or purchase transactions',
-              showActions: false
+              action: () => this.$router.push('/list/Tax')
             }
           ]
         },
         {
-          label: 'Sales',
+          label: _('Sales'),
 
           items: [
             {
-              label: 'Add Items',
+              key: 'Add Sales Items',
+              label: _('Add Items'),
               icon: 'item',
               description:
                 'Add products or services that you sell to your customers',
-              showActions: false
+              action: () => this.$router.push('/list/Item'),
+              fieldname: 'itemCreated'
             },
             {
-              label: 'Add Customers',
+              key: 'Add Customers',
+              label: _('Add Customers'),
               icon: 'customer',
               description: 'Add a few customers to create your first invoice',
-              showActions: false
+              action: () => this.$router.push('/list/Customer'),
+              fieldname: 'customerCreated'
             },
             {
-              label: 'Create Invoice',
+              key: 'Create Invoice',
+              label: _('Create Invoice'),
               icon: 'sales-invoice',
               description:
                 'Create your first invoice and mail it to your customer',
-              showActions: false
+              action: () => this.$router.push('/list/SalesInvoice'),
+              fieldname: 'invoiceCreated'
             }
           ]
         },
         {
-          label: 'Purchase',
+          label: _('Purchase'),
 
           items: [
             {
-              label: 'Add Items',
+              key: 'Add Purchase Items',
+              label: _('Add Items'),
               icon: 'item',
               description:
                 'Add products or services that you buy from your suppliers',
-              showActions: false
+              action: () => this.$router.push('/list/Item'),
+              fieldname: 'itemCreated'
             },
             {
-              label: 'Add Suppliers',
+              key: 'Add Suppliers',
+              label: _('Add Suppliers'),
               icon: 'supplier',
               description: 'Add a few suppliers to create your first bill',
-              showActions: false
+              action: () => this.$router.push('/list/Supplier'),
+              fieldname: 'supplierCreated'
             },
             {
-              label: 'Create Bill',
+              key: 'Create Bill',
+              label: _('Create Bill'),
               icon: 'purchase-invoice',
               description:
                 'Create your first bill and mail it to your supplier',
-              showActions: false
+              action: () => this.$router.push('/list/PurchaseInvoice'),
+              fieldname: 'billCreated'
             }
           ]
         }
-      ]
+      ];
+    }
+  },
+  data() {
+    return {
+      activeCard: null
     };
   },
+  async activated() {
+    frappe.GetStarted = await frappe.getSingle('GetStarted');
+    this.checkForCompletedTasks();
+  },
   methods: {
-    getIconComponent(name) {
+    async checkForCompletedTasks() {
+      let toUpdate = {};
+      if (frappe.GetStarted.onboardingCompleted) {
+        return;
+      }
+
+      let printSettings = await frappe.getSingle('PrintSettings');
+      if (printSettings.logo && printSettings.address) {
+        toUpdate.invoiceSetup = 1;
+      }
+
+      if (!frappe.GetStarted.itemCreated) {
+        let { count } = (
+          await frappe.db.knex('Item').count('name as count')
+        )[0];
+        if (count > 0) {
+          toUpdate.itemCreated = 1;
+        }
+      }
+
+      if (!frappe.GetStarted.invoiceCreated) {
+        let { count } = (
+          await frappe.db.knex('SalesInvoice').count('name as count')
+        )[0];
+        if (count > 0) {
+          toUpdate.invoiceCreated = 1;
+        }
+      }
+
+      if (!frappe.GetStarted.customerCreated) {
+        let { count } = (
+          await frappe.db
+            .knex('Party')
+            .where('customer', 1)
+            .count('name as count')
+        )[0];
+        if (count > 0) {
+          toUpdate.customerCreated = 1;
+        }
+      }
+
+      if (!frappe.GetStarted.billCreated) {
+        let { count } = (
+          await frappe.db.knex('PurchaseInvoice').count('name as count')
+        )[0];
+        if (count > 0) {
+          toUpdate.billCreated = 1;
+        }
+      }
+
+      if (!frappe.GetStarted.supplierCreated) {
+        let { count } = (
+          await frappe.db
+            .knex('Party')
+            .where('supplier', 1)
+            .count('name as count')
+        )[0];
+        if (count > 0) {
+          toUpdate.supplierCreated = 1;
+        }
+      }
+
+      await frappe.GetStarted.update(toUpdate);
+      frappe.GetStarted = await frappe.getSingle('GetStarted');
+    },
+    isCompleted(item) {
+      return frappe.GetStarted.get(item.fieldname) || 0;
+    },
+    getIconComponent(item) {
+      let completed = frappe.GetStarted[item.fieldname] || 0;
+      let name = completed ? 'green-check' : item.icon;
+      let size = completed ? '24' : '18';
       return {
         name,
         render(h) {
@@ -185,7 +315,7 @@ export default {
             props: Object.assign(
               {
                 name,
-                size: '18'
+                size
               },
               this.$attrs
             )
diff --git a/src/pages/SetupWizard/setupCompany.js b/src/pages/SetupWizard/setupCompany.js
index 643a4b57..e7540264 100644
--- a/src/pages/SetupWizard/setupCompany.js
+++ b/src/pages/SetupWizard/setupCompany.js
@@ -37,6 +37,7 @@ export default async function setupCompany(setupWizardValues) {
   await setupChartOfAccounts(bankName);
   await setupRegionalChanges(country);
 
+  await frappe.GetStarted.update({ systemSetup: 1, companySetup: 1 });
   await accountingSettings.update({ setupComplete: 1 });
   frappe.AccountingSettings = accountingSettings;
 }