| | This adds JSON output to the rbt review command. The output contains
|
| | information relevant to the review, such as:
|
| | - The review ID.
|
| | - The review API URL.
|
| | - The review URL.
|
| | - The review status which can be draft , published , or discarded .
|
| | - The review request ID.
|
| | - The review request URL. |
| |
|
| | When a comment subcommand is used, the following comment details are also
|
| | included in the output:
|
| | - The comment type which can be diff_comment , file_attachment_comment ,
|
| | or general_comment .
|
| | - The comment ID.
|
| | - The comment API URL. |
| |
|
~ | | During this change, a regression in Review Board 5.0's API was discovered
|
~ | | which broke the reviews URI template on the Root List Resource. While this
|
~ | | issue is being tracked and dealt with separately, this change avoids
|
~ | | accessing the reviews URI template through the root resource, and instead
|
~ | | accesses the reviews through the review request resource since we're fetching
|
~ | | the review request anyways in order to include its URL in the output. |
| ~ | During testing, a regression in Review Board 5.0's API was discovered
|
| ~ | which broke the reviews URI template on the Root List Resource. This breaks
|
| ~ | the command in cases where a new review draft needs to be created. While the
|
| ~ | API regression is being tracked and dealt with separately, this change allows
|
| ~ | the command to work normally by accessing the reviews through the review
|
| ~ | request resource instead of the root since we're fetching the review request
|
| + | resource anyways in order to include its URL in the output. |
| + |
|
| + | An upcoming change will add unit tests for the rbt review command. |