Skip to content

Commit 16e93f8

Browse files
authored
Merge pull request #622 from Sage-Bionetworks/dev-add-asset-view-to-validation
Always pass asset view to the validation endpoint. This will enable b…
2 parents 8caa1ad + cde7134 commit 16e93f8

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

server.R

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -810,11 +810,12 @@ shinyServer(function(input, output, session) {
810810
.access_token <- access_token
811811
.data_model_labels <- dcc_config_react()$schematic$global$data_model_labels
812812
# asset view must be NULL to avoid cross-manifest validation.
813+
# however, it is necessary for filename validation
814+
# always pass asset view, let the data model decide which validations are run
813815
# doing this in a verbose way to avoid warning with ifelse
814-
.asset_view <- NULL
816+
.asset_view <- selected$master_asset_view()
815817
if (!is.null(dcc_config_react()$schematic$model_validate$enable_cross_manifest_validation) &
816818
isTRUE(dcc_config_react()$schematic$model_validate$enable_cross_manifest_validation)) {
817-
.asset_view <- selected$master_asset_view()
818819
.project_scope <- selected$project()
819820
}
820821

0 commit comments

Comments
 (0)