##// END OF EJS Templates
api: pull-requests get_all now sortes the results and have a flag to show/hide merge result state
marcink -
r3445:f33ce258 default
parent child Browse files
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