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.

Sign up for GitHub

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

Jump to bottom

Allow falsey get parameter values #257

Merged

SleeplessOne1917 merged 1 commit into LemmyNet:main from matc-pub:falsey_get_params

Apr 15, 2024

Contributor

matc-pub commented Apr 15, 2024

0, false and "" currently become None on the server.

Noticed this trying to list comments with max_depth: 0.


          Allow falsey get parameter values

198c7af

matc-pub requested review from dessalines and SleeplessOne1917 as code owners

April 15, 2024 12:16

SleeplessOne1917 approved these changes

View reviewed changes

Member

SleeplessOne1917 commented Apr 15, 2024

Good catch!

SleeplessOne1917 merged commit 0923a67 into LemmyNet:main

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet