Merge pull request #8938 from jakemcdermott/fix-8936

Don't build job redirect until data available

Reviewed-by: https://github.com/apps/softwarefactory-project-zuul
This commit is contained in:
softwarefactory-project-zuul[bot]
2020-12-17 15:24:40 +00:00
committed by GitHub
2 changed files with 7 additions and 4 deletions

View File

@@ -18,8 +18,11 @@ function JobTypeRedirect({ id, path, view, i18n }) {
request: loadJob, request: loadJob,
} = useRequest( } = useRequest(
useCallback(async () => { useCallback(async () => {
const { data } = await UnifiedJobsAPI.read({ id }); const {
return { job: data }; data: { results },
} = await UnifiedJobsAPI.read({ id });
const [item] = results;
return { job: item };
}, [id]), }, [id]),
{ job: {} } { job: {} }
); );
@@ -42,7 +45,7 @@ function JobTypeRedirect({ id, path, view, i18n }) {
</PageSection> </PageSection>
); );
} }
if (isLoading) { if (isLoading || !job?.id) {
// TODO show loading state // TODO show loading state
return <div>Loading...</div>; return <div>Loading...</div>;
} }

View File

@@ -35,7 +35,7 @@ function Templates({ i18n }) {
[`${templatePath}/notifications`]: i18n._(t`Notifications`), [`${templatePath}/notifications`]: i18n._(t`Notifications`),
[`${templatePath}/completed_jobs`]: i18n._(t`Completed Jobs`), [`${templatePath}/completed_jobs`]: i18n._(t`Completed Jobs`),
[surveyPath]: i18n._(t`Survey`), [surveyPath]: i18n._(t`Survey`),
[`${surveyPath}add`]: i18n._(t`Add Question`), [`${surveyPath}/add`]: i18n._(t`Add Question`),
[`${surveyPath}/edit`]: i18n._(t`Edit Question`), [`${surveyPath}/edit`]: i18n._(t`Edit Question`),
[schedulesPath]: i18n._(t`Schedules`), [schedulesPath]: i18n._(t`Schedules`),
[`${schedulesPath}/add`]: i18n._(t`Create New Schedule`), [`${schedulesPath}/add`]: i18n._(t`Create New Schedule`),