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

Fix javascript error when an anonymous user visiting migration page (#32144) #32179

Merged

Conversation

lunny
Copy link
Member

@lunny lunny commented Oct 2, 2024

backport #32144

This PR fixes javascript errors when an anonymous user visits the migration page.
It also makes task view checking more restrictive.

The router moved from /user/task/{id}/status to
/username/reponame/-/migrate/status because it's a migrate status.

…o-gitea#32144)

This PR fixes javascript errors when an anonymous user visits the
migration page.
It also makes task view checking more restrictive.

The router moved from `/user/task/{id}/status` to
`/username/reponame/-/migrate/status` because it's a migrate status.

---------

Co-authored-by: wxiaoguang <[email protected]>
@lunny lunny added the type/bug label Oct 2, 2024
@lunny lunny added this to the 1.22.3 milestone Oct 2, 2024
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Oct 2, 2024
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 2, 2024
@github-actions github-actions bot added modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files modifies/js labels Oct 2, 2024
@manishramanandi
Copy link

Hey i can solve this problem and want to contribute. Can you please explain me more about it?

@lunny
Copy link
Member Author

lunny commented Oct 3, 2024

Hey i can solve this problem and want to contribute. Can you please explain me more about it?

This is a backport pull request. Do you want to review it?

@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 Oct 3, 2024
@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 Oct 4, 2024
@techknowlogick techknowlogick enabled auto-merge (squash) October 4, 2024 17:28
@techknowlogick techknowlogick added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Oct 4, 2024
@techknowlogick techknowlogick merged commit 2e3a191 into go-gitea:release/v1.22 Oct 4, 2024
26 checks passed
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Oct 4, 2024
@lunny lunny deleted the lunny/fix_js_error_migrating2 branch October 4, 2024 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/go Pull requests that update Go code modifies/js modifies/templates This PR modifies the template files size/L Denotes a PR that changes 100-499 lines, ignoring generated files. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants