fix unread count for topics with all messages deleted

Author: or-else

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "tinode-webapp",
   "description": "Tinode messenger for the web",
-  "version": "0.23.0",
+  "version": "0.23.1",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/tinode/webapp.git"
@@ -72,7 +72,7 @@
     "react": "^18.2.0",
     "react-dom": "^18.2.0",
     "react-intl": "^6.4.4",
-    "tinode-sdk": "^0.23.0",
+    "tinode-sdk": "^0.23.1",
     "webm-duration-fix": "^1.0.4"
   },
   "devDependencies": {
@@ -84,11 +84,11 @@
     "babel-plugin-formatjs": "^10.5.3",
     "browserslist": "^4.21.10",
     "copy-webpack-plugin": "^11.0.0",
-    "cssnano": "^6.0.1",
-    "glob": "^10.3.3",
+    "cssnano": "^7.0.0",
+    "glob": "^11.0.0",
     "postcss": "^8.4.27",
-    "postcss-cli": "^10.1.0",
+    "postcss-cli": "^11.0.0",
     "webpack": "^5.88.2",
-    "webpack-cli": "^5.1.4"
+    "webpack-cli": "^6.0.0"
   }
 }

package.json

@@ -1,3 +1,3 @@
 // This is a generated file. Don't edit.
 
-export const PACKAGE_VERSION = "0.23.0";
+export const PACKAGE_VERSION = "0.23.1";

src/version.js

@@ -605,8 +605,8 @@ class MessagesView extends React.Component {
   }
 
   postReadNotification(seq) {
-    // Ignore notifications if the app is invisible.
-    if (!this.props.applicationVisible) {
+    // Ignore notifications if the app is invisible or topic is not yet available.
+    if (!this.props.applicationVisible || !this.state.topic) {
       return;
     }
 
@@ -633,7 +633,9 @@ class MessagesView extends React.Component {
           if (n.sendAt <= now) {
             // Remove expired notification from queue.
             this.readNotificationQueue.shift();
-            seq = Math.max(seq, n.seq);
+            if (n.seq == 0 || Tinode.isServerAssignedSeq(n.seq)) {
+              seq = Math.max(seq, n.seq);
+            }
           } else {
             break;
           }
@@ -643,7 +645,11 @@ class MessagesView extends React.Component {
         if (seq >= 0) {
           const topic = this.props.tinode.getTopic(this.state.topic);
           if (topic) {
-            topic.noteRead(seq);
+            try {
+              topic.noteRead(seq);
+            } catch (err) {
+              console.error("Failed to send read notification", err);
+            }
           }
         }
       }, NOTIFICATION_EXEC_INTERVAL);
@@ -732,10 +738,17 @@ class MessagesView extends React.Component {
 
   handleAllMessagesReceived(count) {
     this.setState({fetchingMessages: false});
-    if (count > 0) {
-      // 0 means "latest".
-      this.postReadNotification(0);
+    if (!count) {
+      // All messages received. Nothing left to receive.
+      const topic = this.props.tinode.getTopic(this.state.topic);
+      if (topic) {
+        // Send read notification for the absolute last message.
+        this.postReadNotification(topic.seq);
+      }
+      return;
     }
+    // 0 means "latest received".
+    this.postReadNotification(0);
   }
 
   handleInfoReceipt(info) {