Aman Agrawal b7e5b7f193 do_update_messages: Delete user_message if guest user loses access. ...

Fixes #14498.
When a topic is moved to a different stream, the message may no
longer be reachable to guest user, if the user is not subscribed
to the new stream.

We used to send message update event to the client in these cases,
which seems to be confusing both to the client updating the message
and the server sending push_notifications for it.

Now, we delete the UserMessage entry for these messages for the
user and send a delete message event to the client; which makes
both push_notification and the event handling client think that
the message was deleted and hence no confusion in the code is
raised.

2020-06-18 01:00:04 -07:00

249 KiB

Raw Blame History

View Raw