From 8a0c3c16ed0d56e45bd17afc4ff2131bf26ca0a5 Mon Sep 17 00:00:00 2001
From: Trent Larson <trent@trentlarson.com>
Date: Tue, 4 Apr 2023 13:06:52 -0600
Subject: [PATCH] add confirmation before confirming hours, plus some other
 verbiage

---
 project.yaml                     |  1 -
 src/views/ContactAmountsView.vue |  8 ++++++--
 src/views/ContactsView.vue       | 17 ++++++++++++-----
 3 files changed, 18 insertions(+), 8 deletions(-)

diff --git a/project.yaml b/project.yaml
index 395944b..1880bd8 100644
--- a/project.yaml
+++ b/project.yaml
@@ -6,7 +6,6 @@
 - replace user-affecting console.logs with error messages (eg. catches)
 
 - contacts v1 :
-  - produce a video assignee:trent
   - .2 warn about amounts when you cannot see them
   - .1 remove 'copy' until it works
   - .5 switch to prod server
diff --git a/src/views/ContactAmountsView.vue b/src/views/ContactAmountsView.vue
index e422c18..1369ce8 100644
--- a/src/views/ContactAmountsView.vue
+++ b/src/views/ContactAmountsView.vue
@@ -72,7 +72,7 @@
                 <fa icon="circle-check" class="text-green-600 fa-fw ml-1" />
                 <span class="tooltiptext">Confirmed</span>
               </span>
-              <button v-else class="tooltip" @click="confirm(record)">
+              <button v-else class="tooltip" @click="confirmGiven(record)">
                 <fa icon="circle" class="text-blue-600 fa-fw ml-1" />
                 <span class="tooltiptext">Unconfirmed</span>
               </button>
@@ -239,7 +239,11 @@ export default class ContactsView extends Vue {
     }
   }
 
-  async confirm(record: GiveServerRecord) {
+  async confirmGiven(record: GiveServerRecord) {
+    if (!confirm("Are you sure you want to mark this as confirmed?")) {
+      return;
+    }
+
     // Make claim
     // I use clone here because otherwise it gets a Proxy object.
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
diff --git a/src/views/ContactsView.vue b/src/views/ContactsView.vue
index 2669fbd..50282d4 100644
--- a/src/views/ContactsView.vue
+++ b/src/views/ContactsView.vue
@@ -166,7 +166,11 @@
                   /* eslint-enable prettier/prettier */
                 }}
                 <span class="tooltiptext-left">
-                  {{ givenByMeDescriptions[contact.did] || "Nothing" }}
+                  {{
+                    givenByMeDescriptions[contact.did]
+                      ? "Most recently: " + givenByMeDescriptions[contact.did]
+                      : "(None given yet.)"
+                  }}
                 </span>
                 <button
                   class="text-md uppercase bg-slate-500 text-white px-1.5 py-2 rounded-md mb-6"
@@ -188,7 +192,11 @@
                   /* eslint-enable prettier/prettier */
                 }}
                 <span class="tooltiptext-left">
-                  {{ givenToMeDescriptions[contact.did] || "Nothing" }}
+                  {{
+                    givenToMeDescriptions[contact.did]
+                      ? "Most recently: " + givenToMeDescriptions[contact.did]
+                      : "(None received yet.)"
+                  }}
                 </span>
                 <button
                   class="text-md uppercase bg-slate-500 text-white px-1.5 py-2 rounded-md mb-6"
@@ -324,8 +332,8 @@ export default class ContactsView extends Vue {
         const contactUnconfirmed: Record<string, number> = {};
         const allData: Array<GiveServerRecord> = resp.data.data;
         for (const give of allData) {
-          if (give.unit == "HUR") {
-            const recipDid: string = give.recipientDid;
+          const recipDid: string = give.recipientDid;
+          if (recipDid && give.unit == "HUR") {
             if (give.amountConfirmed) {
               const prevAmount = contactConfirmed[recipDid] || 0;
               contactConfirmed[recipDid] = prevAmount + give.amount;
@@ -339,7 +347,6 @@ export default class ContactsView extends Vue {
             }
           }
         }
-        //console.log("Done retrieving gives", contactConfirmed);
         this.givenByMeDescriptions = contactDescriptions;
         this.givenByMeConfirmed = contactConfirmed;
         this.givenByMeUnconfirmed = contactUnconfirmed;