Show More
@@ -141,7 +141,8 b' def get_pull_request(request, apiuser, p' | |||
|
141 | 141 | |
|
142 | 142 | |
|
143 | 143 | @jsonrpc_method() |
|
144 |
def get_pull_requests(request, apiuser, repoid, status=Optional('new') |
|
|
144 | def get_pull_requests(request, apiuser, repoid, status=Optional('new'), | |
|
145 | merge_state=Optional(True)): | |
|
145 | 146 | """ |
|
146 | 147 | Get all pull requests from the repository specified in `repoid`. |
|
147 | 148 | |
@@ -155,6 +156,9 b' def get_pull_requests(request, apiuser, ' | |||
|
155 | 156 | * ``open`` |
|
156 | 157 | * ``closed`` |
|
157 | 158 | :type status: str |
|
159 | :param merge_state: Optional calculate merge state for each repository. | |
|
160 | This could result in longer time to fetch the data | |
|
161 | :type merge_state: bool | |
|
158 | 162 | |
|
159 | 163 | Example output: |
|
160 | 164 | |
@@ -232,8 +236,10 b' def get_pull_requests(request, apiuser, ' | |||
|
232 | 236 | validate_repo_permissions(apiuser, repoid, repo, _perms) |
|
233 | 237 | |
|
234 | 238 | status = Optional.extract(status) |
|
235 | pull_requests = PullRequestModel().get_all(repo, statuses=[status]) | |
|
236 | data = [pr.get_api_data() for pr in pull_requests] | |
|
239 | merge_state = Optional.extract(merge_state, binary=True) | |
|
240 | pull_requests = PullRequestModel().get_all(repo, statuses=[status], | |
|
241 | order_by='id', order_dir='desc') | |
|
242 | data = [pr.get_api_data(with_merge_state=merge_state) for pr in pull_requests] | |
|
237 | 243 | return data |
|
238 | 244 | |
|
239 | 245 |
@@ -167,6 +167,7 b' class PullRequestModel(BaseModel):' | |||
|
167 | 167 | if order_by: |
|
168 | 168 | order_map = { |
|
169 | 169 | 'name_raw': PullRequest.pull_request_id, |
|
170 | 'id': PullRequest.pull_request_id, | |
|
170 | 171 | 'title': PullRequest.title, |
|
171 | 172 | 'updated_on_raw': PullRequest.updated_on, |
|
172 | 173 | 'target_repo': PullRequest.target_repo_id |
General Comments 0
You need to be logged in to leave comments.
Login now