Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

404 error when Pull to repo with 'pull requests' disabled #24165

Closed
barzog opened this issue Apr 17, 2023 · 3 comments · Fixed by #24183
Closed

404 error when Pull to repo with 'pull requests' disabled #24165

barzog opened this issue Apr 17, 2023 · 3 comments · Fixed by #24183
Labels
Milestone

Comments

@barzog
Copy link

barzog commented Apr 17, 2023

Description

When we fork repo - forked repo is created with 'pull request' repo setting enabled. Old repo 'pull request' setting is left untouched. When we trying to create pull request from new repo to old repo we receive 404 'Not found' error if 'pull request' setting is disabled in old repo settings. More informative message is needed, or (more better) only repos with 'pull requests' enabled should shown in 'merge into' drop-down list.

Gitea Version

1.19.1

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

image

Git Version

No response

Operating System

Flatcar

How are you running Gitea?

Kubernetes helm-chart on k3s running on flatcar linux.

Database

PostgreSQL

@lunny
Copy link
Member

lunny commented Apr 18, 2023

So should we allow repo to be forked if pull request siabled? 🤔

@barzog
Copy link
Author

barzog commented Apr 18, 2023

This may be even better (but I think that fork button should be shown anyway, but either in 'grayed' state - to show end-user that functionality is there but disabled or in 'normal' state with warning that 'you should enable PR on this repo beforehand.

@silverwind
Copy link
Member

silverwind commented Apr 18, 2023

So should we allow repo to be forked if pull request siabled? 🤔

Absolutely not. Not all forks are for the purpose as a source of pull requests, they might become separate projects etc.

jolheiser pushed a commit that referenced this issue Apr 19, 2023
…) (#24210)

Backport #24183 by @lunny

Fix #24165

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants