Merge pull request #473 from btomaj/main

Fix Remix redirect on bounce

.changeset/tender-rings-invite.md

@@ -0,0 +1,5 @@
+---
+'@shopify/shopify-app-remix': patch
+---
+
+Fixed the errorBoundary to work with new cases in Remix v2.

packages/shopify-app-remix/src/server/boundary/__tests__/error.test.tsx

@@ -13,6 +13,16 @@ describe('Error boundary', () => {
     );
   });
 
+  it('returns a string when handling an ErrorResponseImpl', () => {
+    // WHEN
+    const result = boundary.error(new ErrorResponseImpl());
+
+    // THEN
+    expect(result).toEqual(
+      <div dangerouslySetInnerHTML={{__html: 'Handling response'}} />,
+    );
+  });
+
   it('throws an error when handling an unknown error', () => {
     // WHEN
     const result = () => boundary.error(new Error());
@@ -23,3 +33,4 @@ describe('Error boundary', () => {
 });
 
 class ErrorResponse extends Error {}
+class ErrorResponseImpl extends Error {}

packages/shopify-app-remix/src/server/boundary/error.tsx

@@ -1,5 +1,8 @@
 export function errorBoundary(error: any) {
-  if (error.constructor.name === 'ErrorResponse') {
+  if (
+    error.constructor.name === 'ErrorResponse' ||
+    error.constructor.name === 'ErrorResponseImpl'
+  ) {
     return (
       <div
         dangerouslySetInnerHTML={{__html: error.data || 'Handling response'}}