From da871bfe7a35b66c091233f65b5218f4aa53b15f Mon Sep 17 00:00:00 2001
From: MartinBraquet <martin.braquet@gmail.com>
Date: Mon, 11 May 2026 12:42:19 +0200
Subject: [PATCH] Update profile visibility toggle: replace modal with direct
 API call and toast notifications; remove unused `setShowVisibilityModal` prop

---
 web/components/profile/profile-header.tsx | 33 +++++++++++++++--------
 web/components/profile/profile-info.tsx   |  2 --
 2 files changed, 22 insertions(+), 13 deletions(-)

diff --git a/web/components/profile/profile-header.tsx b/web/components/profile/profile-header.tsx
index 8566073b..3b5f80ae 100644
--- a/web/components/profile/profile-header.tsx
+++ b/web/components/profile/profile-header.tsx
@@ -23,6 +23,7 @@ import {linkClass} from 'web/components/widgets/site-link'
 import {StarButton} from 'web/components/widgets/star-button'
 import {Tooltip} from 'web/components/widgets/tooltip'
 import {useUser} from 'web/hooks/use-user'
+import {updateProfile} from 'web/lib/api'
 import {useT} from 'web/lib/locale'
 import {track} from 'web/lib/service/analytics'
 import {disableProfile} from 'web/lib/util/disable'
@@ -175,17 +176,8 @@ export function ProfileHeaderActions(props: {
   showMessageButton: boolean
   refreshProfile: () => void
   isHiddenFromMe: boolean | undefined
-  setShowVisibilityModal: (show: boolean) => void
 }) {
-  const {
-    user,
-    profile,
-    starredUserIds,
-    refreshStars,
-    showMessageButton,
-    refreshProfile,
-    setShowVisibilityModal,
-  } = props
+  const {user, profile, starredUserIds, refreshStars, showMessageButton, refreshProfile} = props
   const currentUser = useUser()
   const isCurrentUser = currentUser?.id === user.id
   const disabled = profile.disabled
@@ -252,7 +244,26 @@ export function ProfileHeaderActions(props: {
                   ) : (
                     <LockClosedIcon className="h-4 w-4" />
                   ),
-                onClick: () => setShowVisibilityModal(true),
+                onClick: async () => {
+                  toast
+                    .promise(
+                      updateProfile({
+                        visibility: profile.visibility === 'member' ? 'public' : 'member',
+                      }),
+                      {
+                        loading: 'Loading...',
+                        success: 'Success!',
+                        error:
+                          'Failed to update profile visibility. Try again later or contact support.',
+                      },
+                    )
+                    .then(() => {
+                      refreshProfile()
+                    })
+                    .catch(() => {
+                      // noop
+                    })
+                },
               },
               {
                 name: disabled
diff --git a/web/components/profile/profile-info.tsx b/web/components/profile/profile-info.tsx
index 610c3f61..e4e903ef 100644
--- a/web/components/profile/profile-info.tsx
+++ b/web/components/profile/profile-info.tsx
@@ -103,8 +103,6 @@ export function ProfileInfo(props: {
               showMessageButton={showMessageButton}
               refreshProfile={refreshProfile}
               isHiddenFromMe={isHiddenFromMe}
-              // no-op: the visibility modal lives inside ProfileHeader below
-              setShowVisibilityModal={() => {}}
             />
           </div>
         </div>