pagure

#2979 Make fork page header link consistent
Merged by pingou. Opened by lsedlar.
lsedlar/pagure fix-fork-header  into  master

Download 2979.patch

The link used to be like USER/NS / REPO or USER/ REPO with the first slash being part of clickable link to user page. With this page no slash is clickable and there are no spaces.

Alternatively there should be spaces everywhere.

Looks fine to me, I'm going to check the unit-tests before merging though

I tried running the tests, but they just seem to hang. I gave up when they did not finish after more than an hour.

There seems to be one test that is affected:
tests/test_pagure_flask_ui_repo.py", line 760, in test_remove_user_self output.data)

It looks like u'<h2 class="repo-name m-b-0">\n <a href="/test">test</a>' should become: u'<h2 class="repo-name m-b-0">\n<a href="/test">test</a>'.

rebased onto 60f91b8c6f68380e6fb0d4e05927c73228d583aa

That particular test should be fixed now.

Tests passed :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

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

© Red Hat, Inc. and others.