flake8
passed.
JSHint
passed.
Add a wizard for managing the installation of Review Board.
Review Request #13477 — Created Jan. 3, 2024 and submitted
The new wizard walks the user through installing Review Board via the
following steps:
Showing an overview of the install process, along with the system and
target version information, and confirming before moving on.Displays details on the Site vs. Install Directories, asks the user
for the install location for the virtualenv, and validates the path.Shows the commands that will be run and then prompts for confirmation
before installing.Performs the install, streaming results and showing progress.
Asks the user if they want to create a new sitedir or import one, and
walks the user through those choices.If requested, executes
rb-site installto help the user install the
site.Finishes up with a success or error result, and any further steps
that may be needed.This makes the install process very simple, and gives users just enough
information to make the right choices.Right now, the wizard is not invoked. That will be handled in the next
change.
Extensively tested the wizard in both interactive and unattended modes.
Tested validation and error handling, and different install state for
installation options. Completed several installs.
| Summary | ID |
|---|---|
| 828b3770c9e9302a5c7472ad4a371a23b51c9982 |
| Description | From | Last Updated |
|---|---|---|
|
I feel like we should specify that this will exit the installer, and that they'd have to continue this part … |
 maubin
|
|
|
We should link to the documentation. |
maubin
|
|
|
We should add another sentence here to refer them to the documentation for setting up a site directory, and link … |
maubin
|
|
|
Per my comments above, if we add specific links to the documentation I think we could get rid of this … |
maubin
|
|
|
line too long (100 > 79 characters) Column: 80 Error code: E501 |
 reviewbot
|
-
-
I feel like we should specify that this will exit the installer, and that they'd have to continue this part of the installation on their own. Could say something like "Press Control-C if you want to exit the installer and setup your Site Directory later on your own.". Also I have a comment below that mentions how we should link to the documentation for setting up a site directory, we should do that here too before the installer exits.
-
-
We should add another sentence here to refer them to the documentation for setting up a site directory, and link to this documentation.
-
Per my comments above, if we add specific links to the documentation I think we could get rid of this generic "Refer to the documentation to finish setting up" sentence. That way we're not repeating ourselves, and people will be able to easily find what the next steps are, depending on whether the site directory was created or not. We should still keep the "Contact [support_link] if you need assistance" though.
- Change Summary:
-
- Added links to the Creating a Review Board Site docs in the "Your Site Directory" summary and in the import/create success/failed instructions.
- Fixed some capitalization for "site directory" -> "Site Directory," for consistency.
- Removed a double support message for imports.
- Commits:
-
Summary ID 69176336ba890fbec9974b545ec027fcff3300e1 828b3770c9e9302a5c7472ad4a371a23b51c9982 - Diff:
-
Revision 2 (+1336 -18)
- Added Files:
