ref(browserHistory): Remove from teamSettings (#82736)

Author: evanpurkhiser

static/app/views/settings/organizationTeams/teamSettings/index.spec.tsx

@@ -57,9 +57,9 @@ describe('TeamSettings', function () {
     );
 
     await waitFor(() =>
-      expect(router.replace).toHaveBeenCalledWith(
-        '/settings/org-slug/teams/new-slug/settings/'
-      )
+      expect(router.replace).toHaveBeenCalledWith({
+        pathname: '/settings/org-slug/teams/new-slug/settings/',
+      })
     );
   });
 
@@ -102,7 +102,7 @@ describe('TeamSettings', function () {
     );
 
     await waitFor(() =>
-      expect(router.replace).toHaveBeenCalledWith('/settings/org-slug/teams/')
+      expect(router.replace).toHaveBeenCalledWith({pathname: '/settings/org-slug/teams/'})
     );
 
     expect(TeamStore.getAll()).toEqual([]);

static/app/views/settings/organizationTeams/teamSettings/index.tsx

@@ -20,9 +20,8 @@ import {IconDelete} from 'sentry/icons';
 import {t, tct} from 'sentry/locale';
 import type {RouteComponentProps} from 'sentry/types/legacyReactRouter';
 import type {Team} from 'sentry/types/organization';
-import {browserHistory} from 'sentry/utils/browserHistory';
-import normalizeUrl from 'sentry/utils/url/normalizeUrl';
 import useApi from 'sentry/utils/useApi';
+import {useNavigate} from 'sentry/utils/useNavigate';
 import useOrganization from 'sentry/utils/useOrganization';
 import PermissionAlert from 'sentry/views/settings/project/permissionAlert';
 
@@ -31,6 +30,7 @@ interface TeamSettingsProps extends RouteComponentProps<{teamId: string}, {}> {
 }
 
 function TeamSettings({team, params}: TeamSettingsProps) {
+  const navigate = useNavigate();
   const organization = useOrganization();
   const api = useApi();
 
@@ -40,16 +40,16 @@ function TeamSettings({team, params}: TeamSettingsProps) {
     updateTeamSuccess(team.slug, resp);
     if (id === 'slug') {
       addSuccessMessage(t('Team name changed'));
-      browserHistory.replace(
-        normalizeUrl(`/settings/${organization.slug}/teams/${resp.slug}/settings/`)
-      );
+      navigate(`/settings/${organization.slug}/teams/${resp.slug}/settings/`, {
+        replace: true,
+      });
     }
   };
 
   const handleRemoveTeam = async () => {
     try {
       await removeTeam(api, {orgId: organization.slug, teamId: params.teamId});
-      browserHistory.replace(normalizeUrl(`/settings/${organization.slug}/teams/`));
+      navigate(`/settings/${organization.slug}/teams/`, {replace: true});
     } catch {
       // removeTeam already displays an error message
     }