From f27b19f86ac164759cbc5bffc6e4156cc9cd54f5 Mon Sep 17 00:00:00 2001
From: ilkin-overleaf <100852799+ilkin-overleaf@users.noreply.github.com>
Date: Mon, 20 Jul 2026 13:37:57 +0300
Subject: [PATCH] Merge pull request #35607 from
 overleaf/ii-share-modal-invited-people-count

Exclude project owner from "X people invited" count in share modal

GitOrigin-RevId: 7f914a09401691b6e6f7299637ee695ed37d05c0
---
 .../components/project-access.tsx             |  2 +-
 .../components/share-modal-body.tsx           |  5 +---
 .../components/share-project-modal.test.tsx   | 24 +++++++++++++++++--
 3 files changed, 24 insertions(+), 7 deletions(-)

diff --git a/services/web/frontend/js/features/share-project-modal/components/project-access.tsx b/services/web/frontend/js/features/share-project-modal/components/project-access.tsx
index 4287fafe7e..7ea3f95ee3 100644
--- a/services/web/frontend/js/features/share-project-modal/components/project-access.tsx
+++ b/services/web/frontend/js/features/share-project-modal/components/project-access.tsx
@@ -216,7 +216,7 @@ function ProjectAccess({
         <div className="d-inline-flex align-items-center h5 m-0 gap-2">
           <MaterialIcon type="group" unfilled />
           <div className="px-2 fw-normal">
-            {invitedPeopleCount > 1
+            {invitedPeopleCount > 0
               ? t('x_people_invited', { count: invitedPeopleCount })
               : t('no_one_invited_yet')}
           </div>
diff --git a/services/web/frontend/js/features/share-project-modal/components/share-modal-body.tsx b/services/web/frontend/js/features/share-project-modal/components/share-modal-body.tsx
index 344e49ba98..c435405dcd 100644
--- a/services/web/frontend/js/features/share-project-modal/components/share-modal-body.tsx
+++ b/services/web/frontend/js/features/share-project-modal/components/share-modal-body.tsx
@@ -145,10 +145,7 @@ export default function ShareModalBody({
           ) : (
             <ProjectAccess
               setIsInvitedPeopleScreen={setIsInvitedPeopleScreen}
-              // adding +1 for the project owner
-              invitedPeopleCount={
-                sortedMembers.length + (invites || []).length + 1
-              }
+              invitedPeopleCount={sortedMembers.length + (invites || []).length}
             />
           )}
         </>
diff --git a/services/web/test/frontend/features/share-project-modal/components/share-project-modal.test.tsx b/services/web/test/frontend/features/share-project-modal/components/share-project-modal.test.tsx
index 6e01310e43..3d695dfcea 100644
--- a/services/web/test/frontend/features/share-project-modal/components/share-project-modal.test.tsx
+++ b/services/web/test/frontend/features/share-project-modal/components/share-project-modal.test.tsx
@@ -1128,7 +1128,7 @@ describe('<ShareProjectModal/>', function () {
         expect(screen.queryByText('1 person invited')).to.be.null
       })
 
-      it('shows the invited people count, including the owner, when there are collaborators', async function () {
+      it('shows the invited people count, excluding the owner, when there are collaborators', async function () {
         const members: ProjectMember[] = [
           {
             _id: 'member-author' as UserId,
@@ -1151,7 +1151,27 @@ describe('<ShareProjectModal/>', function () {
           createContextProps({ publicAccessLevel: 'private', members })
         )
 
-        await screen.findByText('3 people invited')
+        await screen.findByText('2 people invited')
+        expect(screen.queryByText('No one invited yet')).to.be.null
+      })
+
+      it('shows "1 person invited" when there is a single collaborator', async function () {
+        const members: ProjectMember[] = [
+          {
+            _id: 'member-author' as UserId,
+            email: 'member-author@example.com',
+            privileges: 'readAndWrite',
+            first_name: 'Member',
+            last_name: 'Author',
+          },
+        ]
+
+        renderWithEditorContext(
+          <ShareProjectModal {...modalProps} />,
+          createContextProps({ publicAccessLevel: 'private', members })
+        )
+
+        await screen.findByText('1 person invited')
         expect(screen.queryByText('No one invited yet')).to.be.null
       })
     })