Skip to content

SeenManager and Unseen default feed minds#3092

Manish requested to merge feat/seen-manager-minds-3092 into master

Ticket(s) / Related Merge Requests

issue: minds#3092 (closed)

front MR: front!1885 (merged)

Summary of Changes

  • adds a new SeenManager which uses the logic of UnseenTopFeedManager
  • Adds an unseen param to the FeedsElasticSearchManager->getList() which uses SeenManager to filter out seen posts
  • Removes UnseenTopFeedManager and uses the FeedsElasticSearchManager->getList() with the unseen param

Testing Considerations

use front!1885 (merged) (discovery top and top feed) to test

Deployment Considerations

N/A

Regression Scope

  • top feed is affected

Platform Affected (web, mobile, etc)

N/A

Developer Testing Completed

tested

Screenshots / Screen Recording

...

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Manish

Merge request reports