Allow fromName to be null

src/db/migrations/20210314175439_fromName_null.ts

@@ -0,0 +1,17 @@
+import { Knex } from "knex";
+
+export async function up(knex: Knex) {
+  return knex.transaction((trx) =>
+    trx.schema.alterTable("voicemails", (table) => {
+      table.string("fromName", 64).alter();
+    })
+  );
+}
+
+export async function down(knex: Knex) {
+  return knex.transaction((trx) =>
+    trx.schema.alterTable("voicemails", (table) => {
+      table.string("fromName", 64).notNullable().alter();
+    })
+  );
+}

src/index.ts

@@ -68,20 +68,24 @@ async function initRingCentralSDK() {
     clientId: process.env.RC_APP_KEY,
     clientSecret: process.env.RC_APP_SECRET,
   });
+
+  const login = () =>
+    sdk.login({
+      username: process.env.RC_LOGIN_USERNAME,
+      extension: process.env.RC_LOGIN_EXT,
+      password: process.env.RC_LOGIN_PASSWORD,
+    });
+
   const platform = sdk.platform();
-  platform.on(platform.events.refreshError, (err) => {
+  platform.on(platform.events.refreshError, async (err) => {
-    console.error("Refresh error", err);
+    console.error("Refresh token error:", err);
-  });
+    await login();
-  const client = sdk.client();
+    console.log("RingCentral re-authentication successful.");
-  client.on(client.events.beforeRequest, (request) => {
-    console.log(request.url);
-  });
-  await sdk.login({
-    username: process.env.RC_LOGIN_USERNAME,
-    extension: process.env.RC_LOGIN_EXT,
-    password: process.env.RC_LOGIN_PASSWORD,
   });
+
+  await login();
   console.log("Authenticated to RingCentral.");
+
   return sdk;
 }
 

src/template.tsx

@@ -5,19 +5,21 @@ import { DateTime } from "luxon";
 import type { Contact } from "./types";
 import type { StoredVoicemail, StoredRecording } from "knex/types/tables";
 
-export function getTicketSubject(
+function fromName(vm: StoredVoicemail, contact?: Contact) {
-  voicemail: StoredVoicemail,
+  return contact?.name ?? vm.fromName ?? "unknown";
-  contact?: Contact
+}
-) {
+
-  return `New Voicemail from ${getNationalNumber(voicemail.fromNumber)} (${
+export function getTicketSubject(vm: StoredVoicemail, contact?: Contact) {
-    contact ? contact.name : voicemail.fromName
+  const name = fromName(vm, contact);
-  })`;
+  return `New Voicemail from ${getNationalNumber(vm.fromNumber)} (${name})`;
 }
 
 export function getTicketBody(
   vm: StoredVoicemail & StoredRecording,
   contact?: Contact
 ) {
+  const name = fromName(vm, contact);
+
   return ReactDOMServer.renderToStaticMarkup(
     <div>
       <div>
@@ -25,8 +27,7 @@ export function getTicketBody(
         {DateTime.fromISO(vm.received).toLocaleString(DateTime.DATETIME_MED)}
       </div>
       <div>
-        <b>From:</b> {getNationalNumber(vm.fromNumber)} (
+        <b>From:</b> {getNationalNumber(vm.fromNumber)} ({name})
-        {contact?.name ?? vm.fromName})
       </div>
       <div>
         <b>To:</b> {getNationalNumber(vm.toNumber)}x{vm.extensionNumber} (