Next

Closes #435 (closed) Depends on !347 (merged)

Summary

This page was using some very old code, so I've upgraded it to use the paging tokens. This should avoid situations like this: https://www.minds.com/kratommotivation1/subscriptions

Testing

To test, pick a user with some subscriptions, and visit /{user}/subscriptions.

Click through subscriptions and subscribers, scroll up and down. https://fix-subscription-feed-435.minds.io/

For bonus points, deploy against live data and check: https://www.minds.com/kratommotivation1/subscriptions

added Squad::Yellow scoped label

added 1 commit

• 0f12af4c - Updated

Compare with previous version

unmarked as a Work In Progress

changed the description

added MR::Awaiting Review scoped label

added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label

resolved all threads

added 44 commits

• 0f12af4c...142d395f - 41 commits from branch master
• 1aa0d7f1 - Updated for feedback
• 7de7bcb6 - Merge branch 'master' of gitlab.com:minds/engine into fix/subscription-feed-435
• b0e688e8 - Updated tests

Compare with previous version

resolved all threads

added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label

Getting infinite loops at https://revolver.minds.com/sillysealion5/subscriptions

added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label

mentioned in merge request front!599 (merged)

changed the description

added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label

resolved all threads

Should be fixed with !347 (merged)

approved this merge request

merged

mentioned in commit 4300e1a6