[release-12.0.1] Provisioning: Fix requires migration condition (#104436)

Provisioning: Fix requires migration condition (#104434)

* Provisioning: Fix requires migration condition

* Better error messaging

(cherry picked from commit 82c291675b)

Co-authored-by: Alex Khomenko <Clarity-89@users.noreply.github.com>
This commit is contained in:
grafana-delivery-bot[bot]
2025-04-24 07:48:02 +01:00
committed by GitHub
co-authored by Alex Khomenko
parent ed408985fa
commit 2fbb2d6f5d
2 changed files with 3 additions and 3 deletions
@@ -17,7 +17,7 @@ export function JobContent({ job, isFinishedJob = false }: JobContentProps) {
return null;
}
const { state, message, progress, summary } = job.status;
const { state, message, progress, summary, errors } = job.status;
const repoName = job.metadata?.labels?.['provisioning.grafana.app/repository'];
const getStatusDisplay = () => {
@@ -35,7 +35,7 @@ export function JobContent({ job, isFinishedJob = false }: JobContentProps) {
severity="error"
title={t('provisioning.job-status.status.title-error-running-job', 'Error running job')}
>
{message}
{message ?? errors?.join('\n')}
</Alert>
);
}
@@ -57,7 +57,7 @@ export function BootstrapStep({ onOptionSelect, settingsData, repoName, onStepSt
useEffect(() => {
const { target } = options[0];
setValue('repository.sync.target', target);
onOptionSelect(target !== 'folder' || resourceCount > 0);
onOptionSelect(settingsData?.legacyStorage || resourceCount > 0);
// Only run this effect on mount
// eslint-disable-next-line react-hooks/exhaustive-deps
}, []);