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

hotfix - fix generational animal group form #983

Merged
merged 2 commits into from
Feb 5, 2024
Merged

Conversation

shapiromatron
Copy link
Owner

Fix change in how Django 5 handles choice fields so that the choices are sliced directly instead of the form model field choices, which no longer seems to work. I did a quick keyword search in the codebase and didn't see any other cases where this may have been an issue.

Also wrote a test to test the behavior on the views, and raised code coverage for the forms and views:

image

Copy link
Collaborator

@rabstejnek rabstejnek left a comment

Choose a reason for hiding this comment

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

Tested it out, looks good!

@shapiromatron shapiromatron merged commit bb3e8a2 into main Feb 5, 2024
3 checks passed
@shapiromatron shapiromatron deleted the ag-hotfix branch February 5, 2024 21:16
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