Add tags list for repos whose release setting is disabled by Zettat123 · Pull Request #23465 · go-gitea/gitea

Zettat123 · 2023-03-14T08:31:53Z

If a repo's release setting is enabled, the logic has't changed. Clicking the "Tags" button will jump to /{user}/{repo}/tags and templates/repo/release/list.tmpl template will be used.

If the release setting is disabled, clicking the "Tags" button will still jump to /{user}/{repo}/tags but a new template templates/repo/tag/list.tmpl will be used.

Since both templates above need to render the tags list, I moved the tags list to a shared template located in templates/repo/tag/table.tmpl.

yp05327 · 2023-03-28T04:58:56Z

There are some big changes in #23243, conflicts must be resolved.

GiteaBot · 2023-04-25T16:29:11Z

I was unable to create a backport for 1.19. @Zettat123, please send one manually. 🍵

silverwind · 2023-04-25T16:30:33Z

Sould we really backport such a large PR?

* giteaofficial/main: Fix broken clone script on an empty archived repo (go-gitea#24339) Improve RSS (go-gitea#24335) Automatically select the org when click create repo from org dashboard (go-gitea#24325) on schedule on schedule switch to use Actions from drone for cron (go-gitea#24314) [skip ci] Updated translations via Crowdin Restore bold on repolist (go-gitea#24337) Fix template function DateTime (go-gitea#24317) Fix incorrect CORS response in Http Git handler (go-gitea#24303) Updated upgrade script that is informing user that Gitea service has to be running in order to upgrade it (go-gitea#24260) Add tags list for repos whose release setting is disabled (go-gitea#23465) Refactor config provider (go-gitea#24245) Add RSS Feeds for branches and files (go-gitea#22719) Make SVG in dropdown menu have the same margin-right as IMG (go-gitea#24316) Respect the REGISTER_MANUAL_CONFIRM setting when registering via OIDC (go-gitea#24035)

lunny · 2023-04-26T07:35:11Z

Because this fixed a bug when disabling release, clicking tags on Code tab will return 404.

@wxiaoguang

…3465) Close go-gitea#23427 Co-Author: @wxiaoguang If a repo's release setting is enabled, the logic has't changed. Clicking the "Tags" button will jump to `/{user}/{repo}/tags` and `templates/repo/release/list.tmpl` template will be used. <img src="https://user-images.githubusercontent.com/15528715/224939362-bd8974fd-08b0-4f79-a114-3389d15847ca.png" width="600px" /> If the release setting is disabled, clicking the "Tags" button will still jump to `/{user}/{repo}/tags` but a new template `templates/repo/tag/list.tmpl` will be used. <img src="https://user-images.githubusercontent.com/15528715/233834564-74741e49-f4e9-47c8-ac12-e306642798dc.png" width="600px" /> Since both templates above need to render the tags list, I moved the tags list to a shared template located in `templates/repo/tag/table.tmpl`. --------- Co-authored-by: wxiaoguang <wxiaoguang@gmail.com> Co-authored-by: Giteabot <teabot@gitea.io>

@wxiaoguang

…24369) Backport #23465 Close #23427 Co-Author: @wxiaoguang If a repo's release setting is enabled, the logic has't changed. Clicking the "Tags" button will jump to `/{user}/{repo}/tags` and `templates/repo/release/list.tmpl` template will be used. <img src="https://user-images.githubusercontent.com/15528715/224939362-bd8974fd-08b0-4f79-a114-3389d15847ca.png" width="600px" /> If the release setting is disabled, clicking the "Tags" button will still jump to `/{user}/{repo}/tags` but a new template `templates/repo/tag/list.tmpl` will be used. <img src="https://user-images.githubusercontent.com/15528715/233834564-74741e49-f4e9-47c8-ac12-e306642798dc.png" width="600px" /> Since both templates above need to render the tags list, I moved the tags list to a shared template located in `templates/repo/tag/table.tmpl`.

wxiaoguang · 2023-04-29T03:33:46Z

Because this fixed a bug when disabling release, clicking tags on Code tab will return 404.

I never thought that this would be backported.

It's not a serious bug indeed. Maybe next time we do not need to backport such a big PR.

silverwind · 2023-04-29T06:25:22Z

Yes, I had concerns above, but they were ignored.

lunny · 2023-04-29T09:55:10Z

I don't think so. The bug is if you disable releasing, then you cannot list tags anymore, and the page return 404.

This caused by #23465

This caused by go-gitea#23465

@lunny

Backport #25624 by @lunny This casused by #23465 Before release disabled <img width="1320" alt="图片" src="https://github.com/go-gitea/gitea/assets/81045/190a1c81-daa5-41bc-91ac-c9a0bf629b5f"> release enabled <img width="1320" alt="图片" src="https://github.com/go-gitea/gitea/assets/81045/a0372c31-727c-4ee0-a6b9-30e502498d90"> After release disabled <img width="1304" alt="图片" src="https://github.com/go-gitea/gitea/assets/81045/a747ea80-a3d9-4792-8f6d-e8955da78b9e"> release enabled <img width="1290" alt="图片" src="https://github.com/go-gitea/gitea/assets/81045/7c0bc43a-9149-4148-859d-35839aeb60ca"> Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>

GiteaBot · 2023-08-01T16:01:10Z

We lock pull requests 3 months after they were closed. If there's any need for further discussion, please open a new issue. 🍵

Zettat123 added the type/bug label Mar 14, 2023

lunny reviewed Mar 14, 2023

View reviewed changes

Comment thread routers/web/repo/release.go Outdated

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

lunny reviewed Mar 14, 2023

View reviewed changes

Comment thread routers/web/web.go Outdated

Zettat123 changed the title ~~Add tags list for repos whose release setting is disabled~~ WIP: Add tags list for repos whose release setting is disabled Mar 14, 2023

Zettat123 changed the title ~~WIP: Add tags list for repos whose release setting is disabled~~ Add tags list for repos whose release setting is disabled Mar 15, 2023

Zettat123 requested a review from lunny March 15, 2023 03:42

This comment was marked as off-topic.

Sign in to view

lunny approved these changes Mar 15, 2023

View reviewed changes

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 Mar 15, 2023

lunny added the outdated/backport/v1.19 This PR should be backported to Gitea 1.19 label Mar 15, 2023

lunny added this to the 1.20.0 milestone Mar 15, 2023

yp05327 reviewed Mar 27, 2023

View reviewed changes

Comment thread routers/web/repo/release.go Outdated

Zettat123 force-pushed the bugfix/issue-23427 branch from 6a4ae6f to bd2eefd Compare March 27, 2023 10:39

Zettat123 requested a review from yp05327 March 27, 2023 11:03

yp05327 reviewed Mar 28, 2023

View reviewed changes

Comment thread routers/web/repo/release.go Outdated

Zettat123 force-pushed the bugfix/issue-23427 branch 2 times, most recently from e17756d to 202009c Compare March 28, 2023 05:56

Zettat123 requested a review from yp05327 March 28, 2023 06:05

wxiaoguang reviewed Mar 28, 2023

View reviewed changes

Comment thread templates/repo/sub_menu.tmpl Outdated

wxiaoguang reviewed Mar 28, 2023

View reviewed changes

Comment thread templates/repo/tag/table.tmpl Outdated

Zettat123 requested review from wxiaoguang and removed request for yp05327 March 28, 2023 06:19

Zettat123 force-pushed the bugfix/issue-23427 branch from e2e85a0 to 0ce0f64 Compare March 29, 2023 05:40

add tag list

deafc1a

Zettat123 force-pushed the bugfix/issue-23427 branch from 0ce0f64 to deafc1a Compare March 30, 2023 08:57

wxiaoguang reviewed Apr 23, 2023

View reviewed changes

Comment thread templates/repo/tag/list.tmpl Outdated

wxiaoguang reviewed Apr 23, 2023

View reviewed changes

Comment thread routers/web/repo/release.go Outdated

fix g-modal-confirm

df06a4d

lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Apr 25, 2023

Merge branch 'main' into bugfix/issue-23427

39103f2

silverwind merged commit 30c1cd9 into go-gitea:main Apr 25, 2023

GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Apr 25, 2023

GiteaBot added the backport/manual No power to the bots! Create your backport yourself! label Apr 25, 2023

Zettat123 mentioned this pull request Apr 27, 2023

Add tags list for repos whose release setting is disabled (#23465) #24369

Merged

wxiaoguang deleted the bugfix/issue-23427 branch April 27, 2023 03:26

silverwind mentioned this pull request Apr 28, 2023

getting the tag list does not require being signed in #24413

Merged

wxiaoguang mentioned this pull request May 9, 2023

RSS icon fixes #24476

Merged

lunny mentioned this pull request Jul 2, 2023

Fix tags header and pretty format numbers #25624

Merged

techknowlogick pushed a commit that referenced this pull request Jul 5, 2023

Fix tags header and pretty format numbers (#25624)

90b3b3d

This caused by #23465

GiteaBot mentioned this pull request Jul 5, 2023

Fix tags header and pretty format numbers (#25624) #25694

Merged

GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Jul 5, 2023

Fix tags header and pretty format numbers (go-gitea#25624)

8471c9b

This caused by go-gitea#23465

Zettat123 mentioned this pull request Jul 13, 2023

Improvements of releases list and tags list #25859

Merged

lunny added the backport/done All backports for this PR have been created label Jul 26, 2023

go-gitea locked as resolved and limited conversation to collaborators Aug 1, 2023

Uh oh!

Conversation

Zettat123 commented Mar 14, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

This comment was marked as off-topic.

Uh oh!

Uh oh!

yp05327 commented Mar 28, 2023

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

GiteaBot commented Apr 25, 2023

Uh oh!

silverwind commented Apr 25, 2023

Uh oh!

lunny commented Apr 26, 2023

Uh oh!

wxiaoguang commented Apr 29, 2023

Uh oh!

silverwind commented Apr 29, 2023

Uh oh!

lunny commented Apr 29, 2023

Uh oh!

GiteaBot commented Aug 1, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Zettat123 commented Mar 14, 2023 •

edited

Loading