mattermost / mattermost-mobile

Next generation iOS and Android apps for Mattermost in React Native
https://about.mattermost.com/
Apache License 2.0
2.11k stars 1.28k forks source link

Properly unset the active channel in the server #7919

Closed larkox closed 1 month ago

larkox commented 1 month ago

Summary

When we mark a channel as read on the server, the "active channel" gets set. This handles how we receive push notifications.

This PR makes sure we unset the active channel when we exit from the channel screen, or we send the app to the background.

I also piggybacked a fix for channels in other servers being read when they should not.

Ticket Link

Fix https://mattermost.atlassian.net/browse/MM-57000

Release Note

Improve status management to avoid missing notifications
Fix issue where channels from other servers may get read on startup
enahum commented 1 month ago

Is this PR still waiting for QA?

larkox commented 1 month ago

@enahum Yes. @yasserfaraazkhan friendly reminder.