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

#28357 -- Prepopulated_fields doesn't work for admin.StackedInline #11221

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@jkhlr
Copy link

commented Apr 13, 2019

This PR fixes #28357, which was caused by duplicated logic.
The first commit fixes the bug without removing the duplicated code.
The second commit removes the duplicated code and refactors inline.js (removing ~50 loc)

Fixed #28357 -- Fixed broken prepopulation in StackedInline admin.
Co-authored-by: Florian Wendelborn <1133858+FlorianWendelborn@users.noreply.github.com>

@jkhlr jkhlr force-pushed the jkhlr:master branch 2 times, most recently from bb94e29 to dfb4ab0 Apr 13, 2019

Refactor -- Removed duplicated logic for inline formset creation.
Co-authored-by: Florian Wendelborn <1133858+FlorianWendelborn@users.noreply.github.com>

@jkhlr jkhlr force-pushed the jkhlr:master branch from dfb4ab0 to c8c05a0 Apr 13, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.