Bring the admin settings forms up to modern coding and doc standards.

Review Request #10797 — Created Nov. 11, 2019 and submitted

Information

Review Board
release-4.0.x
e576955...

Reviewers

Our admin settings forms consisted of a mix of new and very old code,
and because of this we had inconsistencies in how some styling and logic
was done. In order to help future-proof things, to help others work on
settings code in a consistent form, and to generally clean up the
codebase, this change updates all of the code to better fit our modern
code and doc standards.

While mostly centered around adding docs and moving from double quotes
to single quotes, this also moves some forms to module-level logging,
updates a URL to the Review Board support settings page, localizes some
strings, moves a complex call to send_mail to use keyword instead of
positional arguments, and improves a check in the SSH settings form.

Unit tests pass.

Went through each form and tested them.

Description From Last Updated

There's a bunch of other couchdb stuff in this file. Should we just get rid of all of it?

daviddavid
david
  1. 
      
  2. reviewboard/admin/forms/storage_settings.py (Diff revision 1)
     
     
     

    There's a bunch of other couchdb stuff in this file. Should we just get rid of all of it?

    1. Looked into what this would take, and we actually have stuff throughout several files for the couchdb "support." I'd rather have a separate change that removed all that (but am all for doing it).

  3. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.0.x (41fbac3)
Loading...