Skip to content

Fix an issue where changing an organization’s visibility caused problems when users had forked its repositories. (#37324)#37344

Merged
silverwind merged 1 commit intogo-gitea:release/v1.26from
GiteaBot:backport-37324-v1.26
Apr 21, 2026
Merged

Fix an issue where changing an organization’s visibility caused problems when users had forked its repositories. (#37324)#37344
silverwind merged 1 commit intogo-gitea:release/v1.26from
GiteaBot:backport-37324-v1.26

Conversation

@GiteaBot
Copy link
Copy Markdown
Collaborator

Backport #37324 by @lunny

A quick fix #37317


The current behavior for forks when an organization or repository is changed to private differs from GitHub.

On GitHub, when a parent repository becomes private, the fork relationship is removed, which keeps the behavior simple and avoids visibility conflicts.

I think we need a similar solution to handle cases where the parent repository becomes private while a fork remains public and the fork relationship is still preserved.

…ems when users had forked its repositories. (go-gitea#37324)

A quick fix go-gitea#37317

---

The current behavior for forks when an organization or repository is
changed to private differs from GitHub.

On GitHub, when a parent repository becomes private, the fork
relationship is removed, which keeps the behavior simple and avoids
visibility conflicts.

I think we need a similar solution to handle cases where the parent
repository becomes private while a fork remains public and the fork
relationship is still preserved.

---------

Signed-off-by: Lunny Xiao <xiaolunwen@gmail.com>
Co-authored-by: silverwind <me@silverwind.io>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
Co-authored-by: Giteabot <teabot@gitea.io>
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 21, 2026
@GiteaBot GiteaBot added this to the 1.26.1 milestone Apr 21, 2026
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Apr 21, 2026
@silverwind silverwind enabled auto-merge (squash) April 21, 2026 18:57
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Apr 21, 2026
@silverwind silverwind merged commit 291f6cb into go-gitea:release/v1.26 Apr 21, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants