Add window.confirm on thesis deletion to prevent accidents

UniversityOfHelsinkiCS · Mar 21, 2024 · 81adcfd · 81adcfd
1 parent 6d9ffb9
commit 81adcfd
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/frontend/src/containers/Thesis/ThesisList.js b/frontend/src/containers/Thesis/ThesisList.js
@@ -30,6 +30,11 @@ export class ThesisList extends Component {
         }
     }
 
+    handleThesisDelete(thesisId) {
+        if (!window.confirm('Are you sure you want to delete this thesis?')) return
+        this.props.deleteThesis(thesisId)
+    }
+
     toggleThesis = thesis => () => {
         const selectedThesesIds = this.state.selectedThesesIds.includes(thesis.thesisId) ?
             this.state.selectedThesesIds.filter(id => id !== thesis.thesisId) :
@@ -179,7 +184,7 @@ export class ThesisList extends Component {
                                 showButtons={this.props.showButtons}
                                 selectable={this.props.selectable}
                                 selectedThesesIds={this.state.selectedThesesIds}
-                                onDelete={() => this.props.deleteThesis(thesis.thesisId)}
+                                onDelete={() => this.handleThesisDelete(thesis.thesisId)}
                             />
                         ))}
                     </tbody>