Opened 7 years ago

Closed 7 years ago

#5765 closed (fixed)

display non_forms error in edit_inline in newforms-admin

Reported by: Honza_Kral Owned by: xian
Component: contrib.admin Version: newforms-admin
Severity: Keywords: nfa-blocker edit_inline template non_form_errors formset ui
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

With the ability of formsets to validate across forms ( #5353 ), the admin form should probably display these.

Attachments (1)

5765.diff (1.5 KB) - added by Honza_Kral 7 years ago.

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by Honza_Kral

comment:1 Changed 7 years ago by jkocherhans

  • Keywords ui added
  • Needs documentation unset
  • Needs tests unset
  • Owner changed from nobody to xian
  • Patch needs improvement unset

This probably needs to be done, but I'll leave it to xian. He may have some other ideas about where to display them, and how to format them.

comment:2 Changed 7 years ago by brosner

  • Keywords nfa-blocker added; newforms admin removed

This does look like something that needs to get done before a merge. Tagging with nfa-blocker.

comment:3 Changed 7 years ago by brosner

  • Triage Stage changed from Unreviewed to Accepted

comment:4 Changed 7 years ago by brosner

  • Resolution set to fixed
  • Status changed from new to closed

(In [7609]) newforms-admin: Fixed #5765 -- non_form_errors now display properly for inline
formsets. Thanks Honza Kral.

Note: See TracTickets for help on using tickets.
Back to Top