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

Change length_is to length == filter. #110

Merged
merged 2 commits into from
Aug 26, 2024
Merged

Conversation

adambirds
Copy link
Contributor

This will fix #109 and needs releasing ASAP.

@adambirds
Copy link
Contributor Author

@assem-ch can this be looked at ASAP? Anyone on latest version of Django can't use admin without this fix.

@Nosudrum
Copy link

@adambirds did you test this ? You might need to remove the ' characters around the value, i.e. use length == 1 instead of length == '1', as it is prescribed in the fix you mentioned on #109.

@adambirds
Copy link
Contributor Author

@Nosudrum I have tested it by installing from my own branch and its working fine with this fix.

@assem-ch assem-ch merged commit e5cd2d9 into assem-ch:master Aug 26, 2024
10 checks passed
@adambirds adambirds deleted the patch-1 branch August 26, 2024 16:12
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.

admin pages crash when using jet reboot with django 5.1
3 participants