Merge pull request #630 from devrnt/555-should-be-able-to-call-boot-more-than-once

555 should be able to call boot more than once

Author: devrnt

.changeset/tall-buses-reply.md

@@ -0,0 +1,5 @@
+---
+'react-use-intercom': major
+---
+
+Allow calling "boot" multiple times. Previously it was not possible to invoke "boot" more than once, because of a "isBooted" guard. This behaviour was not in line with the IntercomJS docs, so it has been removed

packages/react-use-intercom/cypress/e2e/boot.ts

@@ -53,4 +53,31 @@ describe('boot', () => {
     cy.get('[data-cy=show]').click();
     cy.get('.intercom-lightweight-app-launcher-icon-open').should('not.exist');
   });
+
+  it('should allow calling `boot` multiple times', () => {
+    cy.get('[data-cy=boot]').click();
+
+    // Wait for the route aliased as 'intercomPing' to respond
+    // without changing or stubbing its response
+    cy.wait('@intercomPing');
+
+    cy.get('.intercom-lightweight-app-launcher-icon-open').should('exist');
+    cy.window().should('have.property', 'Intercom');
+    cy.window().should('have.deep.property', 'intercomSettings', {
+      app_id: 'jcabc7e3',
+    });
+
+    cy.get('[data-cy="boot-seeded"]').click();
+
+    // Wait for the route aliased as 'intercomPing' to respond
+    // without changing or stubbing its response
+    cy.wait('@intercomPing');
+
+    cy.get('.intercom-lightweight-app-launcher-icon-open').should('exist');
+    cy.window().should('have.property', 'Intercom');
+    cy.window().should('have.deep.property', 'intercomSettings', {
+      app_id: 'jcabc7e3',
+      name: 'Russo',
+    });
+  });
 });

packages/react-use-intercom/src/provider.tsx

@@ -57,10 +57,6 @@ export const IntercomProvider: React.FC<
         return;
       }
 
-      if (isBooted.current) {
-        return;
-      }
-
       const metaData: RawIntercomBootProps = {
         app_id: appId,
         ...(apiBase && { api_base: apiBase }),