Skip to content

Fix various legacy problems#37092

Merged
wxiaoguang merged 7 commits intogo-gitea:mainfrom
wxiaoguang:fix-legacy
Apr 3, 2026
Merged

Fix various legacy problems#37092
wxiaoguang merged 7 commits intogo-gitea:mainfrom
wxiaoguang:fix-legacy

Conversation

@wxiaoguang
Copy link
Copy Markdown
Contributor

@wxiaoguang wxiaoguang commented Apr 3, 2026 •
edited
Loading

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 3, 2026
lunny
lunny reviewed Apr 3, 2026
View reviewed changes
Comment thread routers/web/auth/auth.go
@wxiaoguang
Copy link
Copy Markdown
Contributor Author

wxiaoguang commented Apr 3, 2026

UI fix

Old:

Details image

New:

Details image

@wxiaoguang wxiaoguang added this to the 1.26.0 milestone Apr 3, 2026
@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 Apr 3, 2026
@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 Apr 3, 2026
bircni
bircni reviewed Apr 3, 2026
View reviewed changes
Comment thread modules/markup/markdown/markdown_test.go
@wxiaoguang
Copy link
Copy Markdown
Contributor Author

wxiaoguang commented Apr 3, 2026 •
edited
Loading

All done, large enough (why so many bugs .... 🤣 )

New changes all either have tests or enough comments. And I have manually tested

@bircni
Copy link
Copy Markdown
Member

bircni commented Apr 3, 2026

Good to go!

@wxiaoguang wxiaoguang enabled auto-merge (squash) April 3, 2026 11:30
@wxiaoguang wxiaoguang disabled auto-merge April 3, 2026 11:46
@wxiaoguang wxiaoguang enabled auto-merge (squash) April 3, 2026 11:59
@wxiaoguang wxiaoguang merged commit 74060bb into go-gitea:main Apr 3, 2026
26 checks passed
@wxiaoguang wxiaoguang deleted the fix-legacy branch April 3, 2026 12:34
zjjhot added a commit to zjjhot/gitea that referenced this pull request Apr 4, 2026
@zjjhot
Merge branch 'main' into mainzjj
afe90f0 
* main:
  Update `setup-uv` to v8.0.0 (go-gitea#37101)
  Fix various bugs (go-gitea#37096)
  Clean up AppURL, remove legacy origin-url webcomponent (go-gitea#37090)
  Add e2e reaction test, improve accessibility, enable parallel testing (go-gitea#37081)
  Fix various legacy problems (go-gitea#37092)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bircni bircni bircni approved these changes

@lunny lunny lunny approved these changes

Assignees

No one assigned

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Milestone

1.26.0

4 participants

@wxiaoguang @bircni @lunny @GiteaBot