fix: Mark read channel if shouldMarkActiveChannelAsRead switched on

Author: szuperaz

projects/stream-chat-angular/.eslintrc.json

@@ -55,6 +55,7 @@
       "rules": {
         "jsdoc/require-param-type": 0,
         "jsdoc/require-returns-type": 0,
+        "jsdoc/require-returns": 0,
         "jsdoc/newline-after-description": 0,
         "jsdoc/require-param-description": 0,
         "jsdoc/require-param": 2,

projects/stream-chat-angular/src/lib/channel.service.spec.ts

@@ -367,6 +367,10 @@ describe('ChannelService', () => {
     service.setAsActiveChannel(activeChannel);
 
     expect(activeChannel.markRead).not.toHaveBeenCalledWith();
+
+    service.shouldMarkActiveChannelAsRead = true;
+
+    expect(activeChannel.markRead).toHaveBeenCalledWith();
   });
 
   it(`shouldn't make "markRead" call, if user dosen't have 'read-events' capability`, async () => {

projects/stream-chat-angular/src/lib/channel.service.ts

@@ -224,10 +224,6 @@ export class ChannelService<
     url: string,
     channel: Channel<T>
   ) => Promise<void>;
-  /**
-   * If set to false, read events won't be sent as new messages are received
-   */
-  shouldMarkActiveChannelAsRead = true;
   private channelsSubject = new BehaviorSubject<Channel<T>[] | undefined>(
     undefined
   );
@@ -261,6 +257,7 @@ export class ChannelService<
   private usersTypingInThreadSubject = new BehaviorSubject<UserResponse<T>[]>(
     []
   );
+  private _shouldMarkActiveChannelAsRead = true;
 
   private channelListSetter = (
     channels: (Channel<T> | ChannelResponse<T>)[]
@@ -349,6 +346,23 @@ export class ChannelService<
       this.latestMessageDateByUserByChannelsSubject.asObservable();
   }
 
+  /**
+   * If set to false, read events won't be sent as new messages are received. If set to true active channel (if any) will immediately be marked as read.
+   */
+  get shouldMarkActiveChannelAsRead() {
+    return this._shouldMarkActiveChannelAsRead;
+  }
+
+  /**
+   * If set to false, read events won't be sent as new messages are received. If set to true active channel (if any) will immediately be marked as read.
+   */
+  set shouldMarkActiveChannelAsRead(shouldMarkActiveChannelAsRead: boolean) {
+    if (!this._shouldMarkActiveChannelAsRead && shouldMarkActiveChannelAsRead) {
+      this.activeChannelSubject.getValue()?.markRead();
+    }
+    this._shouldMarkActiveChannelAsRead = shouldMarkActiveChannelAsRead;
+  }
+
   /**
    * Sets the given `channel` as active and marks it as read.
    * @param channel