Skip to content

Improve repo commit view#33877

Merged
wxiaoguang merged 4 commits into
go-gitea:mainfrom
wxiaoguang:improve-repo-view
Mar 14, 2025
Merged

Improve repo commit view#33877
wxiaoguang merged 4 commits into
go-gitea:mainfrom
wxiaoguang:improve-repo-view

Conversation

@wxiaoguang

Copy link
Copy Markdown
Contributor

Fix #24623

Major changes:

  1. Redirect /owner/repo/blob/* requests to /owner/repo/src/commit/* (like GitHub)
  2. Add a "view file diff" link (see screenshot below)
  3. Refactor "AssertHTMLElement" to generic, now we can accurately assert existence or number.
  4. Add more tests
Details

image

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 14, 2025
@wxiaoguang wxiaoguang added the type/enhancement An improvement of existing functionality label Mar 14, 2025
Comment thread templates/repo/commits_list.tmpl Outdated
@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 Mar 14, 2025
Comment thread options/locale/locale_en-US.ini Outdated
@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 Mar 14, 2025
@wxiaoguang wxiaoguang enabled auto-merge (squash) March 14, 2025 07:30
@wxiaoguang wxiaoguang merged commit b094f9b into go-gitea:main Mar 14, 2025
@GiteaBot GiteaBot added this to the 1.24.0 milestone Mar 14, 2025
@wxiaoguang wxiaoguang deleted the improve-repo-view branch March 14, 2025 07:49
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Jun 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/enhancement An improvement of existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rework blob endpoint to match the standard convention

4 participants