Mercurial > kallithea
changeset 3389:1c4505e3be5b beta
show flags, and desc sort pull request based on created_date ref #765
author | Marcin Kuzminski <marcin@python-works.com> |
---|---|
date | Wed, 20 Feb 2013 02:15:12 +0100 |
parents | cb40b3f6428c |
children | 4cd84f4f28fb |
files | rhodecode/controllers/admin/settings.py rhodecode/model/pull_request.py rhodecode/templates/admin/users/user_edit_my_account_pullrequests.html |
diffstat | 3 files changed, 17 insertions(+), 4 deletions(-) [+] |
line wrap: on
line diff
--- a/rhodecode/controllers/admin/settings.py Wed Feb 20 01:59:50 2013 +0100 +++ b/rhodecode/controllers/admin/settings.py Wed Feb 20 02:15:12 2013 +0100 @@ -477,12 +477,16 @@ c.my_pull_requests = PullRequest.query()\ .filter(PullRequest.user_id == self.rhodecode_user.user_id)\ + .order_by(PullRequest.created_on.desc())\ .all() - c.participate_in_pull_requests = \ + + c.participate_in_pull_requests = sorted( [x.pull_request for x in PullRequestReviewers.query()\ .filter(PullRequestReviewers.user_id == self.rhodecode_user.user_id)\ - .all()] + .all()], + key=lambda o: o.created_on, reverse=True) + return render('admin/users/user_edit_my_account_pullrequests.html') @NotAnonymous()
--- a/rhodecode/model/pull_request.py Wed Feb 20 01:59:50 2013 +0100 +++ b/rhodecode/model/pull_request.py Wed Feb 20 02:15:12 2013 +0100 @@ -55,7 +55,7 @@ repo = self._get_repo(repo) return PullRequest.query()\ .filter(PullRequest.other_repo == repo)\ - .order_by(PullRequest.created_on)\ + .order_by(PullRequest.created_on.desc())\ .all() def create(self, created_by, org_repo, org_ref, other_repo, other_ref, @@ -78,7 +78,7 @@ self.sa.add(new) Session().flush() #members - for member in reviewers: + for member in set(reviewers): _usr = self._get_user(member) reviewer = PullRequestReviewers(_usr, new) self.sa.add(reviewer) @@ -116,6 +116,7 @@ 'pr_url': pr_url, 'pr_revisions': revisions } + notif.create(created_by=created_by_user, subject=subject, body=body, recipients=reviewers, type_=Notification.TYPE_PULL_REQUEST, email_kwargs=kwargs)
--- a/rhodecode/templates/admin/users/user_edit_my_account_pullrequests.html Wed Feb 20 01:59:50 2013 +0100 +++ b/rhodecode/templates/admin/users/user_edit_my_account_pullrequests.html Wed Feb 20 02:15:12 2013 +0100 @@ -6,6 +6,10 @@ <li> <div style="height: 12px"> <div style="float:left"> + %if pull_request.is_closed(): + <img src="${h.url('/images/icons/lock_go.png')}" title="${_('Closed')}"/> + %endif + <img src="${h.url('/images/icons/flag_status_%s.png' % str(pull_request.last_review_status))}" /> <a href="${h.url('pullrequest_show',repo_name=pull_request.other_repo.repo_name,pull_request_id=pull_request.pull_request_id)}"> ${_('Pull request #%s opened on %s') % (pull_request.pull_request_id, h.fmt_date(pull_request.created_on))} </a> @@ -29,6 +33,10 @@ %for pull_request in c.participate_in_pull_requests: <li> <div style="height: 12px"> + %if pull_request.is_closed(): + <img src="${h.url('/images/icons/lock_go.png')}" title="${_('Closed')}"/> + %endif + <img src="${h.url('/images/icons/flag_status_%s.png' % str(pull_request.last_review_status))}" /> <a href="${h.url('pullrequest_show',repo_name=pull_request.other_repo.repo_name,pull_request_id=pull_request.pull_request_id)}"> ${_('Pull request #%s opened by %s on %s') % (pull_request.pull_request_id, pull_request.author.full_name, h.fmt_date(pull_request.created_on))} </a>