Misc fixes.

Fix notifications for messages in background script. Fix trade timer removal during cleanup, when there is no trade chat timer.
Mephiles · Nov 13, 2023 · ffe81b2 · ffe81b2
1 parent 57dc602
commit ffe81b2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/extension/scripts/background.js b/extension/scripts/background.js
@@ -607,7 +607,7 @@ async function updateUserdata() {
 				let message = `${messages.last().title} - by ${messages.last().name}`;
 				if (messages.length > 1) message += `\n(and ${messages.length - 1} more message${messages.length > 2 ? "s" : ""})`;
 
-				notifications.events.combined = newNotification(`New Message${applyPlural(messages.length)}`, message, LINKS.messages);
+				notifications.messages.combined = newNotification(`New Message${applyPlural(messages.length)}`, message, LINKS.messages);
 			}
 		}
 

diff --git a/extension/scripts/features/trade-timer/ttTradeTimer.js b/extension/scripts/features/trade-timer/ttTradeTimer.js
@@ -99,7 +99,7 @@
 	}
 
 	function cleanup() {
-		timer.remove();
+		timer?.remove();
 		timer = null;
 
 		const tradeChat = getTradeChat();