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

Allow filtering by multiple users #595

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Allow filtering by multiple users #595

wants to merge 2 commits into from

Conversation

TheAMM
Copy link
Contributor

@TheAMM TheAMM commented Oct 31, 2019

⚠️ Merge after #592 ⚠️

...by repeating &u=user in the GET parameters.
No UX for this yet.
Reworks the RSS URL generator to fit with duplicated keys.

Sort of tested but could use a sharp look at.

same_user = False
if logged_in_user:
    same_user = len(user_ids) == 1 and logged_in_user.id in user_ids

should replicate the previous functionality well enough.

SQA's baked queries prepares the queries in advance, caching yada yada.
Makes thing a bit faster.
Also bigger speedup included is a shoddy cache for the total torrent
count (only applied to baked queries currently). Caching the value for a
few dozen seconds shaves off some wasted time, as it's mostly just used
for pagination.
...by repeating &u=user in the GET parameters.
No UX for this yet.
Reworks the RSS URL generator to fit with duplicated keys.
@Vodes
Copy link

Vodes commented Oct 16, 2021

Is this ever going to be a thing? I'd rather not get ratelimited trying to avoid trolls ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants