PR#1631: To get the user's username we need to access the user first, fixes the links - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

pagure

#1631 To get the user's username we need to access the user first, fixes the links

Merged Dec 4, 2016 by pingou. Opened Dec 2, 2016 by pingou.

fix_urls into master

Download 1631.patch

no initial comment

vivekanand1101 commented Dec 2, 2016

:thumbsup:

jcline commented Dec 2, 2016

This will also be a problem in the user_requests.html template

vivekanand1101 commented Dec 2, 2016

This will also be a problem in the user_requests.html template

I guess there were no PRs on stg that's why there was no 500 there

rebased

Dec 2, 2016

jcline commented Dec 2, 2016

:thumbsup:

2 new commits added

Expand the unit-tests on view_user_issues and view_user_requests
Small fix in the html to avoid an almost empty line

Dec 3, 2016

pingou commented Dec 4, 2016

I'm going to assume the :thumbsup: still stand even with the added unit-tests :)

Thanks for the reviews!

rebased

Dec 4, 2016

Pull-Request has been merged by pingou

Dec 4, 2016

Metadata

Assignee

None

Tags

No Tags

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.