Update the SSH Settings page to use the new admin UI styling.

Review Request #10943 — Created March 9, 2020 and submitted

Information

Review Board
release-4.0.x

Reviewers

The SSH Settings page now uses the modern CSS components, making it fit
in with the rest of the administration UI.

There are a few improvements to the text shown in the UI. We now have a
banner that explains where SSH keys are used (helping to avoid confusion
by users who think they're mandatory).

Form handling has also improved, making use of the add_error() method
on forms to set error details, and making use of submit buttons rather
than multiple form elements to set the action being performed (upload,
delete, generate).

Tested uploading keys, generating keys, and deleting keys.

Tested all the possible error conditions, simulating each one to see
the resulting error information.

Unit tests pass.

Summary ID
Update the SSH Settings page to use the new admin UI styling.
The SSH Settings page now uses the modern CSS components, making it fit in with the rest of the administration UI. There are a few improvements to the text shown in the UI. We now have a banner that explains where SSH keys are used (helping to avoid confusion by users who think they're mandatory). Form handling has also improved, making use of the `add_error()` method on forms to set error details, and making use of submit buttons rather than multiple form elements to set the action being performed (upload, delete, generate).
ddc5ba200d8fd17174719f0b1e0162d85c26cbc8

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

Status: Closed (submitted)

Change Summary:

Pushed to release-4.0.x (f37069e)
Loading...