Skip to content

Just mark thread chat read if the feature toggle is off

Tomas Vik requested to merge mark-thread-chat-read into develop

https://gitlab.com/gitlab-org/gitter/webapp/merge_requests/1622#note_236758863

The problem that is being fixed (and testing strategy at the same time):

  1. admin enables threaded conversations
  2. some threaded messages have been sent and users received unread notifications
  3. admin disables threaded conversations

Without this fix, there is going to be unread indicator but clicking it won't do anything.

After this fix clicking the indicator marks the chats read.

This is not optimal but it is fixing an edge case and it's only temporary till we remove the feature toggle.

Merge request reports