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

Incorrect latest commit message in UI when viewing tag #21687

Closed
silverwind opened this issue Nov 4, 2022 · 1 comment · Fixed by #21804
Closed

Incorrect latest commit message in UI when viewing tag #21687

silverwind opened this issue Nov 4, 2022 · 1 comment · Fixed by #21804
Labels

Comments

@silverwind
Copy link
Member

silverwind commented Nov 4, 2022

Description

The latest commit is incorrectly rendered when viewing a tag, the first line (commit title) is absent. It's okay when viewing a branch.

Not OK (tag view) https://try.gitea.io/silverwind/remap-css/src/tag/5.0.4:

image

OK (branch view) https://try.gitea.io/silverwind/remap-css/src/branch/master:

image

Gitea Version

1.19

Can you reproduce the bug on the Gitea demo site?

Yes

@KN4CK3R
Copy link
Member

KN4CK3R commented Nov 5, 2022

Added a patch in #21693.

silverwind added a commit to silverwind/gitea that referenced this issue Nov 13, 2022
It is not correct to return tag data like the tag's message when
commit data requested. This changes fixes commit retrieval by tag for
both the latest commit in the UI and the commit info on tag webhook
events.

Fixes: go-gitea#21687
Replaces: go-gitea#21693
lunny added a commit that referenced this issue Mar 2, 2023
It is not correct to return tag data when commit data is requested, so
remove the hacky code that overwrote parts of a commit with parts of a
tag.

This fixes commit retrieval by tag for both the latest commit in the UI
and the commit info on tag webhook events.

Fixes: #21687
Replaces: #21693

<img width="324" alt="Screenshot 2022-11-13 at 15 26 37"
src="https://user-images.githubusercontent.com/115237/201526975-736c6ea7-ad6a-467a-a823-9a63d6ecb718.png">

<img width="789" alt="image"
src="https://user-images.githubusercontent.com/115237/201526876-90a13ffc-1e5c-4d76-911b-f1ae51e8eaab.png">

---------

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
GiteaBot pushed a commit to GiteaBot/gitea that referenced this issue Mar 3, 2023
It is not correct to return tag data when commit data is requested, so
remove the hacky code that overwrote parts of a commit with parts of a
tag.

This fixes commit retrieval by tag for both the latest commit in the UI
and the commit info on tag webhook events.

Fixes: go-gitea#21687
Replaces: go-gitea#21693

<img width="324" alt="Screenshot 2022-11-13 at 15 26 37"
src="https://user-images.githubusercontent.com/115237/201526975-736c6ea7-ad6a-467a-a823-9a63d6ecb718.png">

<img width="789" alt="image"
src="https://user-images.githubusercontent.com/115237/201526876-90a13ffc-1e5c-4d76-911b-f1ae51e8eaab.png">

---------

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
jolheiser pushed a commit that referenced this issue Mar 3, 2023
Backport #21804

It is not correct to return tag data when commit data is requested, so
remove the hacky code that overwrote parts of a commit with parts of a
tag.

This fixes commit retrieval by tag for both the latest commit in the UI
and the commit info on tag webhook events.

Fixes: #21687
Replaces: #21693

<img width="324" alt="Screenshot 2022-11-13 at 15 26 37"
src="https://user-images.githubusercontent.com/115237/201526975-736c6ea7-ad6a-467a-a823-9a63d6ecb718.png">

<img width="789" alt="image"
src="https://user-images.githubusercontent.com/115237/201526876-90a13ffc-1e5c-4d76-911b-f1ae51e8eaab.png">

Co-authored-by: silverwind <me@silverwind.io>
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 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.

2 participants