Allow ConfigForms to use fieldsets.

Review Request #9045 - Created June 29, 2017 and submitted

David Trowbridge
Djblets
release-0.10.x
886d3f8...
djblets

This change updates things so that if a ConfigForm has fieldsets, they'll be
rendered correctly (as opposed to just having all the fields rendered together
as if the fieldsets didn't exist).

This has one small niggle in that if a person defines their own fieldsets, one
of those fieldsets must contain the form_target field. I've added a log
statement to help people figure out what went wrong if they didn't include
this.

Testing done:
- Ran unit tests.
- Used this in conjunction with a Review Board change.

  • Ran unit tests.
  • Used this in conjunction with a Review Board change.
  • 0
  • 6
  • 0
  • 6
Description From Last Updated
Christian Hammond
David Trowbridge
Barret Rennie
David Trowbridge
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.10.x (215bbc2)
Loading...