NotificationsYou must be signed in to change notification settings
Fork6.3k
Star52.6k

Fix empty git repo handling logic#33101

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

wxiaoguang merged 2 commits intogo-gitea:mainfromwxiaoguang:fix-repo-empty

Jan 4, 2025

Merged

Fix empty git repo handling logic#33101

wxiaoguang merged 2 commits intogo-gitea:mainfromwxiaoguang:fix-repo-empty

Jan 4, 2025

Conversation

Copy link

Contributor

wxiaoguang commentedJan 4, 2025•
edited
Loading

Fix#33092

GiteaBot added the lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. label

Jan 4, 2025

pull-request-sizebot added the size/M label

Jan 4, 2025

github-actionsbot added modifies/translation modifies/go

Pull requests that update Go code

modifies/templatesThis PR modifies the template files labels

Jan 4, 2025

wxiaoguang mentioned this pull request

Jan 4, 2025

Fix no branch redirection loop#33100

Closed

wxiaoguang commented

Jan 4, 2025

View reviewed changes

templates/repo/empty.tmplShow resolvedHide resolved

wxiaoguang force-pushed thefix-repo-empty branch fromb337dda to4ab5817Compare

January 4, 2025 05:20

fix

dc262dc

wxiaoguang force-pushed thefix-repo-empty branch from4ab5817 todc262dcCompare

January 4, 2025 05:23

lunny approved these changes

Jan 4, 2025

View reviewed changes

GiteaBot added lgtm/need 1This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labels

Jan 4, 2025

update comment

7a3b301

Zettat123 approved these changes

Jan 4, 2025

View reviewed changes

GiteaBot added lgtm/doneThis PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1This PR needs approval from one additional maintainer to be merged. labels

Jan 4, 2025

wxiaoguang merged commit3d544a3 intogo-gitea:main

Jan 4, 2025

26 checks passed

GiteaBot added this to the1.24.0 milestone

Jan 4, 2025

wxiaoguang added the backport/v1.23 label

Jan 4, 2025

wxiaoguang deleted the fix-repo-empty branch

January 4, 2025 10:47

wxiaoguang added a commit to wxiaoguang/gitea that referenced this pull request

Jan 4, 2025

Fix empty git repo handling logic (go-gitea#33101)

411ce55

Fixgo-gitea#33092

wxiaoguang mentioned this pull request

Jan 4, 2025

Fix empty git repo handling logic and fix mobile view (#33101)#33102

Merged

wxiaoguang added backport/done

All backports for this PR have been created

backport/manualNo power to the bots! Create your backport yourself! labels

Jan 4, 2025

wxiaoguang added a commit that referenced this pull request

Jan 5, 2025

Fix empty git repo handling logic and fix mobile view (#33101) (#33102)

e637008

Backport#33101 and UI fix from main (including#33108)

yp05327 reviewed

Jan 6, 2025

View reviewed changes

templates/repo/empty.tmpl

Comment on lines +20 to +21

		{{elseif .Repository.IsEmpty}}
		<divclass="ui segment center">{{ctx.Locale.Tr"repo.no_branch"}}</div>

Copy link

Contributor

yp05327Jan 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

The quick guide will not be displayed any more after creating a new repo.

Copy link

ContributorAuthor

wxiaoguangJan 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Will fix

Copy link

ContributorAuthor

wxiaoguangJan 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

main: Fix repo empty guide#33114

1.23: together in Fix package error handling and npm meta and empty repo guide#33112

And added a test for the guide page to make sure it won't break again.

zjjhot added a commit to zjjhot/gitea that referenced this pull request

Jan 6, 2025

Merge remote-tracking branch 'giteaofficial/main'

6f15516

* giteaofficial/main:  [skip ci] Updated translations via Crowdin  Fix dropdown menu header and mobile view (go-gitea#33108)  workflow_dispatch use workflow from trigger branch (go-gitea#33098)  [skip ci] Updated translations via Crowdin  Trivial fixes (go-gitea#33103)  Fix empty git repo handling logic (go-gitea#33101)  Refactor legacy line-number and scroll code (go-gitea#33094)  Use `Project-URL` metadata field to get a PyPI package's homepage URL (go-gitea#33089)  Clean up legacy form CSS styles (go-gitea#33081)  [skip ci] Updated translations via Crowdin  add myself to maintainers (go-gitea#33088)  Make Gitea always use its internal config, ignore `/etc/gitconfig` (go-gitea#33076)  Fix "stop time tracking button" on navbar (go-gitea#33084)  Fix bleve fuzziness search (go-gitea#33078)  Try to fix ACME directory problem (go-gitea#33072)  Refactor env var related code (go-gitea#33075)