dce-reactkit 5.0.7 → 5.0.8

package/dist/esm/types/helpers/visitServerEndpoint.d.ts

@@ -22,6 +22,8 @@ export declare const _setStubResponse: (opts: {
  * @param opts.path - the path of the server endpoint
  * @param [opts.method=GET] - the method of the endpoint
  * @param [opts.params] - query/body parameters to include
+ * @param [opts.headers] - custom headers to include; values must already be
+ *    string to avoid issue with header serialization
  * @returns response from server
  */
 declare const visitServerEndpoint: (opts: {
@@ -30,5 +32,8 @@ declare const visitServerEndpoint: (opts: {
     params?: {
         [x: string]: any;
     } | undefined;
+    headers?: {
+        [x: string]: string;
+    } | undefined;
 }) => Promise<any>;
 export default visitServerEndpoint;

package/dist/index.d.ts

@@ -865,6 +865,8 @@ declare const initClient: (opts: ({
  * @param opts.path - the path of the server endpoint
  * @param [opts.method=GET] - the method of the endpoint
  * @param [opts.params] - query/body parameters to include
+ * @param [opts.headers] - custom headers to include; values must already be
+ *    string to avoid issue with header serialization
  * @returns response from server
  */
 declare const visitServerEndpoint: (opts: {
@@ -873,6 +875,9 @@ declare const visitServerEndpoint: (opts: {
     params?: {
         [x: string]: any;
     } | undefined;
+    headers?: {
+        [x: string]: string;
+    } | undefined;
 }) => Promise<any>;
 
 /**

package/package.json

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

package/src/components/Tooltip.tsx

@@ -63,6 +63,14 @@ const Tooltip: React.FC<Props> = (props) => {
         // Import bootstrap tooltip
         const BSTooltip = (await import('bootstrap')).Tooltip;
 
+        // The child may have unmounted while the dynamic import above was
+        // resolving (React nulls the ref on unmount). Constructing a
+        // Bootstrap tooltip with a null element causes an error,
+        // so bail out instead
+        if (!childRef.current) {
+          return;
+        }
+
         // Initialize
         t = new BSTooltip(
           childRef.current,
@@ -78,6 +86,15 @@ const Tooltip: React.FC<Props> = (props) => {
       return () => {
         if (t) {
           t.dispose();
+          // Bootstrap's hide() queues a transition-end callback that can
+          // fire AFTER dispose and touch the (now null) trigger map and
+          // element (twbs/bootstrap issue #37474). Point those private
+          // fields at harmless stand-ins so the late callback is a no-op
+          // instead of a crash.
+          /* eslint-disable no-underscore-dangle */
+          t._activeTrigger = {};
+          t._element = document.createElement('noscript');
+          /* eslint-enable no-underscore-dangle */
         }
       };
     },

package/src/helpers/visitServerEndpoint.tsx

@@ -96,6 +96,8 @@ export const _setStubResponse = (
  * @param opts.path - the path of the server endpoint
  * @param [opts.method=GET] - the method of the endpoint
  * @param [opts.params] - query/body parameters to include
+ * @param [opts.headers] - custom headers to include; values must already be
+ *    string to avoid issue with header serialization
  * @returns response from server
  */
 const visitServerEndpoint = async (
@@ -103,6 +105,8 @@ const visitServerEndpoint = async (
     path: string,
     method?: ('GET' | 'POST' | 'DELETE' | 'PUT'),
     params?: { [key in string]: any },
+    // Headers have no serialization layer, so use string
+    headers?: { [key in string]: string },
   },
 ): Promise<any> => {
   // Set default method
@@ -159,6 +163,7 @@ const visitServerEndpoint = async (
     path: opts.path,
     method: opts.method ?? 'GET',
     params,
+    headers: opts.headers,
   });
 
   // Check for failure