From 573769fcc7accdd5db0e93b588ecaf5bf9a30818 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jun 02 2015 06:52:33 +0000 Subject: Redirect the user to the pull-request created after the creation --- diff --git a/pagure/lib/__init__.py b/pagure/lib/__init__.py index 155c84c..2b4db8a 100644 --- a/pagure/lib/__init__.py +++ b/pagure/lib/__init__.py @@ -899,7 +899,7 @@ def new_pull_request(session, repo_from, branch_from, ) ) - return 'Request created' + return request def edit_issue(session, issue, ticketfolder, user, diff --git a/pagure/ui/fork.py b/pagure/ui/fork.py index 7a0ae16..952b7f4 100644 --- a/pagure/ui/fork.py +++ b/pagure/ui/fork.py @@ -688,7 +688,7 @@ def new_request_pull(repo, branch_to, branch_from, username=None): if repo.parent: parent = repo.parent - message = pagure.lib.new_pull_request( + request = pagure.lib.new_pull_request( SESSION, repo_to=parent, branch_to=branch_to, @@ -700,7 +700,7 @@ def new_request_pull(repo, branch_to, branch_from, username=None): ) try: SESSION.commit() - flask.flash(message) + flask.flash('Request created') except SQLAlchemyError as err: # pragma: no cover SESSION.rollback() APP.logger.exception(err) @@ -710,10 +710,12 @@ def new_request_pull(repo, branch_to, branch_from, username=None): if not parent.is_fork: url = flask.url_for( - 'request_pulls', username=None, repo=parent.name) + 'request_pull', requestid=request.id, + username=None, repo=parent.name) else: url = flask.url_for( - 'request_pulls', username=parent.user, repo=parent.name) + 'request_pull', requestid=request.id, + username=parent.user, repo=parent.name) return flask.redirect(url) except pagure.exceptions.PagureException, err: # pragma: no cover