Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add limit to how far back in time anonymous users can browse public TLs #12863

Open
wants to merge 2 commits into
base: master
from

Conversation

@ThibG
Copy link
Collaborator

ThibG commented Jan 13, 2020

Fixes #12858

This limits unauthenticated users to viewing up to the 40 newest toots
in the public TL + any newer than 2 hours ago.

@ThibG ThibG force-pushed the ThibG:fixes/public-timeline-unauthenticated-cutoff branch 4 times, most recently from ad36b16 to c0f872c Jan 13, 2020
Fixes #12858

This limits unauthenticated users to viewing up to the 40 newest toots
in the public TL + any newer than 2 hours ago.
@ThibG ThibG force-pushed the ThibG:fixes/public-timeline-unauthenticated-cutoff branch from c0f872c to d3c61dc Jan 13, 2020
@ykzts
ykzts approved these changes Jan 14, 2020
@Gargron

This comment has been minimized.

Copy link
Member

Gargron commented Jan 15, 2020

It might lead to visitors assuming there is nothing older on the network. How about showing a message like "There is more, but you need to sign up to see it" when the limit is reached?

@ThibG ThibG force-pushed the ThibG:fixes/public-timeline-unauthenticated-cutoff branch from 4652540 to 37f9c3a Jan 15, 2020
@ThibG

This comment has been minimized.

Copy link
Collaborator Author

ThibG commented Jan 15, 2020

Added the following message, which appears only when there's nothing more to load, and no user is logged on.

image

@ThibG ThibG requested a review from Gargron Jan 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.