search

TrangPham / django-admin-confirm

AdminConfirmMixin is a mixin for ModelAdmin that adds confirmations to changes, additions and actions.
Other
129 stars 16 forks source link

Confirmation page for new record shows field names as None #53

Open TrangPham opened 1 year ago

TrangPham commented 1 year ago

Describe the bug

https://github.com/TrangPham/django-admin-confirm/assets/2117313/cbeb1222-38f6-4cc9-97d5-394d8375137b

See recording at 00:25

To Reproduce Steps to reproduce the behavior:

  1. On Admin with confirm_add = True, create new entity that triggers confirmation page
  2. On confirmation page see the diff table, but the field names are None

Expected behavior On Confirmation Page, field names should not be None

Screenshots See above

Versions (please complete the following information):

Additional context Add any other context about the problem here.

dtcooper commented 7 months ago

+1 on this! Poking around to see if I can fix it. :)

dtcooper commented 7 months ago

Submitted a PR for this! See #54.