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

Update CommunitySelectorModal API query to use global limit #1653

Merged

Conversation

kminardo
Copy link
Contributor

@kminardo kminardo commented Nov 4, 2024

Change to correct behavior documented @ #1652

I chose to use the existing LIMIT constant (50) instead of 20 - but can correct this to exactly match lemmy-ui, if that's preferred.

@kminardo kminardo requested a review from aeharding as a code owner November 4, 2024 23:10
Copy link
Owner

@aeharding aeharding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks great! Apologies if it take a while to release, the app is currently in a state of flux :)

@aeharding aeharding merged commit 21e637a into aeharding:main Nov 5, 2024
2 checks passed
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