Testing

• Go to https://goal-top-algorithm-redux-2.minds.io/newsfeed/global/top;period=12h
• Check that the Posts older than 12H legend appears at some point, with older posts below
• Play around with TOP and LATEST filters.

Note: Do not use HOT as it's running on the old votes-per-period system which isn't running on Sandboxes.

Devs: Refreshing Top posts list

• Switch to Sandbox cluster
• $ kubectl get pods | grep goal-top-algorithm-redux-2-minds-app
• $ kubectl exec -ti goal-top-algorithm-redux-2-minds-app-{random_ID_HERE} sh
• $ cd /var/www/Minds/engine
• $ php cli.php top all sync_activity --metric=down --from=1 && php cli.php top all sync_activity --metric=up --from=1 && php cli.php top all sync_images --metric=down --from=1 && php cli.php top all sync_images --metric=up --from=1 && php cli.php top all sync_videos --metric=down --from=1 && php cli.php top all sync_videos --metric=up --from=1 && php cli.php top all sync_blogs --metric=down --from=1 && php cli.php top all sync_blogs --metric=up --from=1

Issues

• Closes #2269 (closed)

added scoped labels

changed milestone to %HILE Q4/2019

added 1 commit

• 637b88fb - (fix): Misaligned margin

Compare with previous version

added 1 commit

• 768c7a5f - (chore): Enable 1Y back

Compare with previous version

changed the description

mentioned in merge request engine!407 (merged)

There's something funky up with the feed selectors.

When I hit this, with the query parameters, it works.

https://goal-top-algorithm-redux-2.minds.io/newsfeed/global/top;period=12h;all=1

When you navigate back to newsfeed and then back, however, you get nothing

Screen capture here https://www.screencast.com/t/G96AUFdUby

@edgebal pointed out that the preferred filter gets set to On when switching to discovery tab. It's not an issue for this card.

approved this merge request

@xander-miller Can you review this MR?

As a heads up, @xander-miller, I liked a couple of media posts and re-ran the sync command. You should be able to use the type filter (all/images/videos/blogs)

changed the description

approved this merge request

@markeharding I don't have merge request rights to merge this in anymore.

assigned to @markeharding

This shouldn't show be showing when the feature flag is off.

added scoped label and automatically removed label

added 1 commit

• 1ea1f841 - (fix): Feature flag for "posts older" divider

Compare with previous version

approved this merge request

merged

mentioned in commit f90c7262