Skip to content

Improve feed refreshing logic

Martin Santangelo requested to merge js/feed-refresh-improvements into develop

Ticket(s) / Related Merge Requests

#5094 (closed)

Summary of Changes

Improved feed refresh logic and performance

  • Tapping the tab scrolls to the top if not there or refresh the feed otherwise

  • Latest feed is only refreshed by pressing the tab if there are new posts (The new post floating button is visible)

  • Refreshing the feed refresh only the current tab, not all of them

  • Refreshing the latest tab now loads the highlights in parallel and waits for them before updating the list, this solves the performance issues when scrolling fast after refreshing

  • Changelog file updated with this template (one or more)

    • Added for new features.

Testing Considerations

Check that it complies with:

  • when down in the feed, tapping on home will scroll to the top.
  • when down in the feed, tapping on new items notification will scroll to top and refresh the feed
  • when at the top of the feed when tapping on home and new items, it will refresh
  • when at the top of the feed when tapping on home and no items, it will do nothing
  • on pull to refresh, it will refresh the the feed no matter what.

Regression Scope

It affects the newsfeed mostly, but the discovery feeds should be checked too.

Platform Affected (web, mobile, etc)

both

Developer Testing Completed

Android device

Screenshots

(Any extra UI screens that can be shared)

Does this impact

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

Merge request reports