FEATURE: Dismiss new and unread for PM inboxes. (PR #13911)

GitHub

The title of this pull request changed from “WIP: Dismiss new and unread for PM inboxes.” to "Dismiss new and unread for PM inboxes.

This pull request introduces 2 alerts when merging a1b5ddd05a6c0270466598952b1d14cc77875256 into 016efeadf6f242e04daf5ef8e18c2ca708a1392d - view on LGTM.com

new alerts:

  • 2 for Overwritten property

This pull request introduces 2 alerts when merging 16f8cbc828babd08ef86753d22cf9e7937dc1c4f into 016efeadf6f242e04daf5ef8e18c2ca708a1392d - view on LGTM.com

new alerts:

  • 2 for Overwritten property

This is only used in user-topics-list so it shouldn’t be here in the first place.

The title of this pull request changed from “Dismiss new and unread for PM inboxes.” to "FEATURE: Dismiss new and unread for PM inboxes.

Very minor but this could be this.set since it’s only one prop.

Excellent PR. It’s so rare that I review something this long with only a trivial comment.

LGTM, nicely done and well-tested

@eviltrout @martin-brennan Thank you for the review :+1: