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

'Commits since this release' link broken #18226

Closed
OCram85 opened this issue Jan 10, 2022 · 0 comments · Fixed by #18252
Closed

'Commits since this release' link broken #18226

OCram85 opened this issue Jan 10, 2022 · 0 comments · Fixed by #18252
Labels
Milestone

Comments

@OCram85
Copy link
Contributor

OCram85 commented Jan 10, 2022

Gitea Version

1.15.9

Git Version

2.30.2

Operating System

SLES 15 SP2

How are you running Gitea?

Running Gitea in this environment:

[2x haproxy / keepalived nodes] --- (ponting to  traefik reverse proxy) ---> [ multiple docker swarm manager] ---- running ---> [ Single gitea container ]

Database

MySQL

Can you reproduce the bug on the Gitea demo site?

Yes. See releases for https://gitea.com/xorm/xorm/releases

Log Gist

No response

Description

Just noticed the individual links on the releases page missing the diff target.

Example: The release v1.0.7 shown in the screenshot indicates 2 commits since this release. When hoovering on the link the second parameter for viewing the diff is missing: http://<gitea-url>/<ORG>/<Repo>/.../v.1.0.7...

  • I guess it should be: http://<gitea-url>/<ORG>/<Repo>/.../v.1.0.7...<next Release tag> or even http://<gitea-url>/<ORG>/<Repo>/.../v.1.0.7...HEAD ?

Could this related to the merging strategy? I always use Squash Merge

Screenshots

image

@OCram85 OCram85 changed the title Commits since this release link broken 'Commits since this release' link broken Jan 10, 2022
@lunny lunny added this to the 1.15.10 milestone Jan 11, 2022
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
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