• 
      

    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)
       
       
       
      Show all issues

      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:
    Completed
    Change Summary:
    Pushed to release-4.0.x (41fbac3)