SeenManager and Unseen default feed minds#3092
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 theFeedsElasticSearchManager->getList()
which uses SeenManager to filter out seen posts - Removes UnseenTopFeedManager and uses the
FeedsElasticSearchManager->getList()
with theunseen
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