feat: add confirmation modal to topic event deletion

v1.18.x
Julian Lam 4 years ago
parent 1ee92c281d
commit e803737a4b

@ -24,6 +24,7 @@
"edit": "Edit", "edit": "Edit",
"delete": "Delete", "delete": "Delete",
"delete-event": "Delete Event", "delete-event": "Delete Event",
"delete-event-confirm": "Are you sure you want to delete this event?",
"purge": "Purge", "purge": "Purge",
"restore": "Restore", "restore": "Restore",
"move": "Move", "move": "Move",

@ -53,12 +53,15 @@ define('forum/topic/threadTools', [
topicContainer.on('click', '[component="topic/event/delete"]', function () { topicContainer.on('click', '[component="topic/event/delete"]', function () {
const eventId = $(this).attr('data-topic-event-id'); const eventId = $(this).attr('data-topic-event-id');
const eventEl = $(this).parents('[component="topic/event"]'); const eventEl = $(this).parents('[component="topic/event"]');
api.del(`/topics/${tid}/events/${eventId}`, {}) bootbox.confirm('[[topic:delete-event-confirm]]', (ok) => {
.then(function () { if (ok) {
eventEl.remove(); api.del(`/topics/${tid}/events/${eventId}`, {})
}) .then(function () {
.catch(app.alertError); eventEl.remove();
return false; })
.catch(app.alertError);
}
})
}); });
// todo: should also use topicCommand, but no write api call exists for this yet // todo: should also use topicCommand, but no write api call exists for this yet

Loading…
Cancel
Save