dce-reactkit 3.2.2-beta.31 → 3.2.2-beta.33

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dce-reactkit",
-  "version": "3.2.2-beta.31",
+  "version": "3.2.2-beta.33",
   "description": "Shared components for Harvard DCE apps",
   "main": "dist/cjs/index.js",
   "module": "dist/esm/index.js",

package/src/components/IntelliTable.tsx

@@ -272,12 +272,14 @@ const IntelliTable: React.FC<Props> = (props) => {
         // Custom info based on current sort type
         let sortButtonAriaLabel;
         let sortIcon = faSort;
+        let sortingByThisColumn = false;
         if (!sortColumnParam) {
           // Not being sorted yet
           sortButtonAriaLabel = `sort ascending by ${column.title}`;
           sortIcon = faSort;
         } else if (column.param === sortColumnParam) {
           // Already sorted by this column
+          sortingByThisColumn = true;
           if (sortType === SortType.Ascending) {
             // Sorted ascending
             sortButtonAriaLabel = `sort descending by ${column.title}`;
@@ -299,6 +301,7 @@ const IntelliTable: React.FC<Props> = (props) => {
             key={column.param}
             scope="col"
             id={`IntelliTable-${id}-header-${column.param}`}
+            className="text-start"
           >
             <div className="d-flex align-items-center justify-content-center flex-row h-100">
               {/* Title */}
@@ -309,7 +312,8 @@ const IntelliTable: React.FC<Props> = (props) => {
               <div>
                 <button
                   type="button"
-                  className="btn btn-light btn-sm ms-1"
+                  id={`IntelliTable-${id}-sort-by-${column.param}-button`}
+                  className={`btn btn-${sortingByThisColumn ? 'light' : 'secondary'} btn-sm ms-1 ps-1 pe-1 pt-0 pb-0`}
                   aria-label={sortButtonAriaLabel}
                   onClick={() => {
                     dispatch({

package/src/components/LogReviewer.tsx

@@ -1961,7 +1961,7 @@ const LogReviewer: React.FC<Props> = (props) => {
         type: ParamType.Int,
       },
       {
-        title: 'Student',
+        title: 'Student?',
         param: 'isLearner',
         type: ParamType.Boolean,
       },

package/src/helpers/genCSV.ts

@@ -5,13 +5,13 @@
  * @returns escaped cell text
  */
 const escapeCellText = (text: string): string => {
-  if (!text.includes(',')) {
+  if (!String(text).includes(',')) {
     // No need to escape
-    return text;
+    return String(text);
   }
 
   // Perform escape
-  return `"${text.replace(/"/g, '""')}`;
+  return `"${String(text).replace(/"/g, '""')}`;
 };
 
 /**
@@ -46,7 +46,24 @@ const genCSV = (
     csv += (
       columns
         .map((column) => {
-          return escapeCellText(datum[column.param]);
+          let contents: string;
+          const cell = datum[column.param];
+          if (
+            typeof cell === 'string'
+            || typeof cell === 'number'
+          ) {
+            contents = String(cell);
+          } else if (
+            typeof cell === 'undefined'
+            || cell === null
+          ) {
+            contents = '';
+          } else if (typeof cell === 'object') {
+            contents = JSON.stringify(cell);
+          } else {
+            contents = '';
+          }
+          return escapeCellText(contents);
         })
         .join(',')
     );

package/src/helpers/genRouteHandler.ts

@@ -500,7 +500,7 @@ const genRouteHandler = (
           isAdmin: !!launchInfo.isAdmin,
           isTTM: !!launchInfo.isTTM,
           courseId: launchInfo.courseId,
-          courseName: launchInfo.courseName,
+          courseName: launchInfo.contextLabel,
           browser,
           device,
           year,