Skip to content

fix: remove duplicate IDs (#35210) #35325

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Aug 22, 2025
Merged

fix: remove duplicate IDs (#35210) #35325

merged 2 commits into from
Aug 22, 2025

Conversation

lunny
Copy link
Member

@lunny lunny commented Aug 21, 2025

backport #35210

@lunny lunny added the type/bug label Aug 21, 2025
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 21, 2025
@github-actions github-actions bot added modifies/translation modifies/api This PR adds API routes or modifies them modifies/go Pull requests that update Go code modifies/cli PR changes something on the CLI, i.e. gitea doctor or gitea admin modifies/templates This PR modifies the template files modifies/docs modifies/migrations modifies/internal modifies/dependencies modifies/frontend labels Aug 21, 2025
@lunny lunny changed the base branch from main to release/v1.24 August 21, 2025 04:06
@lunny lunny removed modifies/translation modifies/api This PR adds API routes or modifies them modifies/go Pull requests that update Go code modifies/cli PR changes something on the CLI, i.e. gitea doctor or gitea admin modifies/docs modifies/migrations modifies/internal modifies/dependencies modifies/frontend labels Aug 21, 2025
@lunny lunny added this to the 1.24.6 milestone Aug 21, 2025
@lunny lunny changed the title Backport 35210 fix: remove duplicate IDs (#35210) Aug 21, 2025
@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 Aug 21, 2025
@GiteaBot GiteaBot removed the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Aug 21, 2025
@GiteaBot GiteaBot added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Aug 21, 2025
@lunny lunny enabled auto-merge (squash) August 21, 2025 17:15
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Aug 21, 2025
@lunny lunny disabled auto-merge August 21, 2025 17:15
@lunny lunny merged commit cc5ccf4 into go-gitea:release/v1.24 Aug 22, 2025
26 checks passed
@lunny lunny deleted the backport_35210 branch August 22, 2025 23:20
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Aug 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@silverwind silverwind silverwind approved these changes

+1 more reviewer

@a1012112796 a1012112796 a1012112796 approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/templates This PR modifies the template files type/bug
Projects
None yet
Milestone
1.24.6
Development

Successfully merging this pull request may close these issues.
4 participants
@lunny @silverwind @a1012112796 @GiteaBot