Fix integration config forms to show local site when fieldsets exist.

Review Request #9604 - Created Feb. 8, 2018 and submitted

David Trowbridge
Review Board
release-3.0.x
a09a8ad...
reviewboard

All integration configs for Review Board automatically include a local
site field. In the case where there were no fieldsets, all the fields
would show up fine, but if the form tried to group things, the local
site field would disappear. This is because the djblets form was
defining the basic info fieldset with only the "name" and "enable"
fields.

I've fixed the Review Board form to properly override the basic info
fieldset, so that local site shows up everywhere.

Loaded some integration config forms that previously exhibited a missing
"Local Site" field. Saw the field appear.

  • 0
  • 0
  • 2
  • 0
  • 2
Description From Last Updated
Christian Hammond
  1. 
      
  2. "Review Board" in the description.

  3. "Testing Done" is sitting in the description.

  4. 
      
David Trowbridge
Barret Rennie
  1. Ship It!
  2. 
      
David Trowbridge
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (9c0ccc7)
Loading...