Skip to content

Remove channel and clustered recs experiments #5822

Olivia Madrid requested to merge chore/cleanup-cluster-rec-exp-5822 into master

Ticket(s) / Related Merge Requests

#5822 (closed)

Summary of Changes

  • Removed wrapper that only showed channel recommendations in the channel feed if the flag was on. Now it should always show on other users' empty channel feeds (i.e. no post has been made by the channel)
  • Changed conditions for when the default feed loads the clustered recs endpoint api/v3/newsfeed/feed/clustered-recommendations (instead of default feed endpoint api/v3/newsfeed/default-feed). Previously it needed a location input AND experiment enabled. Now it just needs a location input to show clustered recs. Note there is only one scenario where the location input is used -- discovery top feed.
  • Found extra padding in guest mode default feed when testing, removed it

Testing Considerations

  • discovery top feed
  • channel feed
  • guest mode default feed (it should NOT be using the clustered recs endpoint)

Deployment Considerations

  1. Merge in tandem with engine!1186 (merged)

  2. Delete minds-2263-clustered-recs and minds-2933-channelrecs from growthbook once merged

Regression Scope

Feeds

Platform Affected (web, mobile, etc)

Web

Developer Testing Completed

See screenshots

Screenshots / Screen Recording

Channel recs on an empty channel feed image

Clustered recs endpoint in discovery top image

Guest mode default feed is not using clustered recs endpoint image

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 Olivia Madrid

Merge request reports