fix: delete project (#3393)

Flagsmith · Feb 8, 2024 · be544e2 · be544e2
1 parent 5f13624
commit be544e2
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 8 deletions.
diff --git a/frontend/common/providers/OrganisationProvider.js b/frontend/common/providers/OrganisationProvider.js
@@ -20,6 +20,10 @@ const OrganisationProvider = class extends Component {
       users: OrganisationStore.getUsers(),
     }
     ES6Component(this)
+    if (props.onRemoveProject) {
+      this.listenTo(OrganisationStore, 'removed', props.onRemoveProject)
+    }
+
     this.listenTo(OrganisationStore, 'change', () => {
       this.setState({
         groups: UserGroupStore.getGroups(),

diff --git a/frontend/web/components/ProjectManageWidget.tsx b/frontend/web/components/ProjectManageWidget.tsx
@@ -67,7 +67,11 @@ const ProjectManageWidget: FC<SegmentsPageType> = ({
   }, [handleCreateProjectClick, router.route.location])
 
   return (
-    <OrganisationProvider>
+    <OrganisationProvider
+      onRemoveProject={() => {
+        toast('Your project has been removed')
+      }}
+    >
       {({
         isLoading,
         projects,

diff --git a/frontend/web/components/pages/ProjectSettingsPage.js b/frontend/web/components/pages/ProjectSettingsPage.js
@@ -77,12 +77,6 @@ const ProjectSettingsPage = class extends Component {
       AppActions.getProject(this.props.match.params.projectId)
     }
   }
-
-  onRemove = () => {
-    toast('Your project has been removed')
-    this.context.router.history.replace('/projects')
-  }
-
   confirmRemove = (project, cb) => {
     openModal(
       'Delete Project',
@@ -163,7 +157,6 @@ const ProjectSettingsPage = class extends Component {
       <div className='app-container container'>
         <ProjectProvider
           id={this.props.match.params.projectId}
-          onRemove={this.onRemove}
           onSave={this.onSave}
         >
           {({ deleteProject, editProject, isLoading, isSaving, project }) => {
@@ -430,6 +423,7 @@ const ProjectSettingsPage = class extends Component {
                           <Button
                             onClick={() =>
                               this.confirmRemove(project, () => {
+                                this.context.router.history.replace('/projects')
                                 deleteProject(this.props.match.params.projectId)
                               })
                             }