Skip to content

Conversation

@zouharvi
Copy link
Contributor

  • Removes @login_required, making campaign-status accessible even without a login. Tested that it still works if someone is logged in and is is_staff.
  • Removes sort_key arguments since it's not being used anywhere.

@snukky
Copy link
Contributor

snukky commented Jul 26, 2025

Technically, sort_key is used if provided in the URL after last /:

r'(?P<sort_key>[0123456])?/?$',

If it works, that's fine for now. I'm unsure if we can drop the login requirement beyond WMT entirely, but let's not bother with it at the moment neither. Thanks!

@snukky snukky merged commit f375cae into develop Jul 26, 2025
3 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.

3 participants