Skip to content

Commit d91383b

Browse files
authored
Merge pull request #21165 from mvdbeek/public_fallback
[25.0] Fix workflow landing rendering if public query param not provided
2 parents b57e5ce + 0c4f318 commit d91383b

2 files changed

Lines changed: 2 additions & 4 deletions

File tree

client/src/components/Dataset/DatasetView.vue

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,7 @@ const showError = computed(
5757
const isAutoDownloadType = computed(
5858
() => dataset.value && datatypeStore.isDatatypeAutoDownload(dataset.value.file_ext)
5959
);
60-
const downloadUrl = computed(
61-
() => withPrefix(`/datasets/${props.datasetId}/display`)
62-
);
60+
const downloadUrl = computed(() => withPrefix(`/datasets/${props.datasetId}/display`));
6361
const preferredVisualization = computed(
6462
() => dataset.value && datatypeStore.getPreferredVisualization(dataset.value.file_ext)
6563
);

client/src/entry/analysis/router.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -560,7 +560,7 @@ export function getRouter(Galaxy) {
560560
component: WorkflowLanding,
561561
props: (route) => ({
562562
uuid: route.params.uuid,
563-
public: route.query.public.toLowerCase() === "true",
563+
public: (route.query.public || "").toLowerCase() === "true",
564564
secret: route.query.client_secret,
565565
}),
566566
beforeEnter: requireAuth,

0 commit comments

Comments
 (0)