Web app: better notifications for job deletion

This commit is contained in:
Sybren A. Stüvel 2023-02-07 15:23:13 +01:00
parent bf464055e0
commit c1070b838e

View File

@ -42,10 +42,18 @@ export const useNotifs = defineStore('notifications', {
* @param {API.SocketIOJobUpdate} jobUpdate Job update received via SocketIO. * @param {API.SocketIOJobUpdate} jobUpdate Job update received via SocketIO.
*/ */
addJobUpdate(jobUpdate) { addJobUpdate(jobUpdate) {
let msg = `Job ${jobUpdate.name}`; let msg = "Job";
if (jobUpdate.previous_status && jobUpdate.previous_status != jobUpdate.status) { if (jobUpdate.name) msg += ` ${jobUpdate.name}`;
if (jobUpdate.was_deleted) {
msg += " was deleted";
}
else if (jobUpdate.previous_status && jobUpdate.previous_status != jobUpdate.status) {
msg += ` changed status ${jobUpdate.previous_status}${jobUpdate.status}`; msg += ` changed status ${jobUpdate.previous_status}${jobUpdate.status}`;
} }
else {
// Don't bother logging just "Job" + its name, as it conveys no info.
return;
}
this.add(msg) this.add(msg)
}, },