From a989404e23f736e7ce1c71e7105506e3bc6cdd76 Mon Sep 17 00:00:00 2001 From: Lunny Xiao Date: Wed, 2 Oct 2024 12:37:16 +0800 Subject: Fix javascript error when an anonymous user visiting migration page (#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 --- web_src/js/features/repo-migrate.ts | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'web_src') diff --git a/web_src/js/features/repo-migrate.ts b/web_src/js/features/repo-migrate.ts index dc36177940..b75289feec 100644 --- a/web_src/js/features/repo-migrate.ts +++ b/web_src/js/features/repo-migrate.ts @@ -1,19 +1,17 @@ import {hideElem, showElem} from '../utils/dom.ts'; import {GET, POST} from '../modules/fetch.ts'; -const {appSubUrl} = window.config; - export function initRepoMigrationStatusChecker() { const repoMigrating = document.querySelector('#repo_migrating'); if (!repoMigrating) return; - document.querySelector('#repo_migrating_retry').addEventListener('click', doMigrationRetry); + document.querySelector('#repo_migrating_retry')?.addEventListener('click', doMigrationRetry); - const task = repoMigrating.getAttribute('data-migrating-task-id'); + const repoLink = repoMigrating.getAttribute('data-migrating-repo-link'); // returns true if the refresh still needs to be called after a while const refresh = async () => { - const res = await GET(`${appSubUrl}/user/task/${task}`); + const res = await GET(`${repoLink}/-/migrate/status`); if (res.status !== 200) return true; // continue to refresh if network error occurs const data = await res.json(); -- cgit v1.2.3