comments are addressed

src/lib/formatters.js

@@ -1,22 +1,11 @@
 export function dateTimeFormat(timestamp, options) {
   const dateTime = new Date(timestamp);
-  const timeFormatOptions = {
-    hour: 'numeric',
-    minute: 'numeric',
-    second: 'numeric',
-  };
-  const time = new Intl.DateTimeFormat(undefined, timeFormatOptions).format(
-    dateTime
-  );
+  const timeFormatOptions = { timeStyle: 'medium' };
+  const dateFormatOptions = { dateStyle: 'medium' };
 
-  if (options?.timeOnly) {
-    return time;
-  }
+  const chosenOption = options?.timeOnly
+    ? timeFormatOptions
+    : { ...timeFormatOptions, ...dateFormatOptions };
 
-  const dateFormatOptions = { month: 'short', day: 'numeric', year: 'numeric' };
-  const date = new Intl.DateTimeFormat(undefined, dateFormatOptions).format(
-    dateTime
-  );
-
-  return `${date} ${time}`;
+  return new Intl.DateTimeFormat(undefined, chosenOption).format(dateTime);
 }

tests/ext-activitylog.test.js

@@ -306,14 +306,13 @@ test('timestamp is formatted and rendered correctly', () => {
   const timeZone = { timeZone: 'UTC' };
   // For log timestamp: 1597686226302
   const expectedTime = '5:43:46 PM';
-  const expectedDateTime = `Aug 17, 2020 5:43:46 PM`;
+  const expectedDateTime = `Aug 17, 2020, 5:43:46 PM`;
 
   const { activityLog } = new ActivityLog();
   // To have consistant date time format, we choose "en-US" date time formatting and UTC timezone.
   IntlDateTimeFormatFn.mockImplementation(function (zone, options) {
     Object.assign(options, timeZone);
-    this.format = (date) =>
-      new originalIntlDateTimeFormat('en-US', options).format(date);
+    return new originalIntlDateTimeFormat('en-US', options);
   });
 
   activityLog.handleNewLogs(logs);