"New pull request" button shown when viewing tag/commit #1339

Closed
opened 2025-11-02 03:56:59 -06:00 by GiteaMirror · 6 comments
Owner

Originally created by @ethantkoenig on GitHub (Dec 10, 2017).

  • Gitea version (or commit ref): master
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (ccd21f0837, click on the "New pull request" button for the 404)
    • No
    • Not relevant

Description

When viewing a tag or commit, the "new pull request" button is shown. If you click on the link, you will get a 404, since the head of a PR must be a branch.

I think that ideally the button should not be shown when viewing a tag/commit.

Originally created by @ethantkoenig on GitHub (Dec 10, 2017). - Gitea version (or commit ref): master - Can you reproduce the bug at https://try.gitea.io: - [x] Yes (https://try.gitea.io/ethantkoenig/sandbox/src/commit/ccd21f083756c783bc4725512edb9992183ff941, click on the "New pull request" button for the 404) - [ ] No - [ ] Not relevant ## Description When viewing a tag or commit, the "new pull request" button is shown. If you click on the link, you will get a 404, since the head of a PR must be a branch. I think that ideally the button should not be shown when viewing a tag/commit.
GiteaMirror added the type/enhancement label 2025-11-02 03:56:59 -06:00
Author
Owner

@Albert221 commented on GitHub (Dec 10, 2017):

I'd like to work on that.

@Albert221 commented on GitHub (Dec 10, 2017): I'd like to work on that.
Author
Owner

@lafriks commented on GitHub (Dec 10, 2017):

@Albert221 I think there is already PR to fix this

@lafriks commented on GitHub (Dec 10, 2017): @Albert221 I think there is already PR to fix this
Author
Owner

@Albert221 commented on GitHub (Dec 10, 2017):

@lafriks could you confirm? I don't see any opened PRs addressing that

@Albert221 commented on GitHub (Dec 10, 2017): @lafriks could you confirm? I don't see any opened PRs addressing that
Author
Owner

@lafriks commented on GitHub (Dec 10, 2017):

@ethantkoenig wrote that in #3133 description that it fixes this issue but in code somehow I don't see this

@lafriks commented on GitHub (Dec 10, 2017): @ethantkoenig wrote that in #3133 description that it fixes this issue but in code somehow I don't see this >
Author
Owner

@ethantkoenig commented on GitHub (Dec 10, 2017):

Oh, I forgot to push my latest changes to #3133. Yes, #3133 will fix this issue

@ethantkoenig commented on GitHub (Dec 10, 2017): Oh, I forgot to push my latest changes to #3133. Yes, #3133 will fix this issue
Author
Owner

@lafriks commented on GitHub (Jan 14, 2018):

Fixed by #3133

@lafriks commented on GitHub (Jan 14, 2018): Fixed by #3133
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#1339