We currently have the api/reviews/all API endpoint that lets us fetch reviews based on their status: expired, pending, active, or merged. However, we are missing one important status: rejected reviews. These are reviews where the pull request (PR) was closed without merging.
Identifying Rejected Reviews
A review can be classified as 'rejected' if it meets all of the following criteria (to be confirmed by @Extheoisah):
- The review has been submitted.
- It has not been merged.
- It has been archived.
Proposed Change
We need to update the api/reviews/all endpoint to include 'rejected' as a status option. This will allow us to track and manage these reviews more effectively, ensuring that all types of review outcomes are visible and accounted for in our system.
We currently have the
api/reviews/allAPI endpoint that lets us fetch reviews based on their status: expired, pending, active, or merged. However, we are missing one important status: rejected reviews. These are reviews where the pull request (PR) was closed without merging.Identifying Rejected Reviews
A review can be classified as 'rejected' if it meets all of the following criteria (to be confirmed by @Extheoisah):
Proposed Change
We need to update the
api/reviews/allendpoint to include 'rejected' as a status option. This will allow us to track and manage these reviews more effectively, ensuring that all types of review outcomes are visible and accounted for in our system.