Skip to content

Fix notifications unread logic #4371 #4377

Martin Santangelo requested to merge fix/notifications-unread-logic into develop

Ticket(s) / Related Merge Requests

#4371 (closed) #4377 (closed)

Summary of Changes

  • Removes refresh when focusing the screen to prevent scroll jumps
  • Removes the set all as unread on focus
  • Fixed the mark as unready logic
  • Checked that the counter is updated by the push notifications

Testing Considerations

  • open the app
  • send many notifications from a secondary account
  • open notifications
  • the unread blue dot should be removed when a notification is visible for a moment

Regression Scope

(Can this change cause another regression issue? Which areas to focus on for regression testing?)

Platform Affected (web, mobile, etc)

both

Developer Testing Completed

Tested on an Android device and the iOS simulator

Screenshots

(WhSimulator_Screen_Recording_-iPhone_13-_2022-08-08_at_15.54.38

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode
Edited by Martin Santangelo

Merge request reports