Skip to content

Commit

Permalink
Always pass asset view to the validation endpoint. This will enable b…
Browse files Browse the repository at this point in the history
…oth cross-manifest validation and filename validation IF the data model supports those rules. This change is to make DCA compatible with filename validation rules.
  • Loading branch information
afwillia committed Oct 3, 2024
1 parent 8caa1ad commit cde7134
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions server.R
Original file line number Diff line number Diff line change
Expand Up @@ -810,11 +810,12 @@ shinyServer(function(input, output, session) {
.access_token <- access_token
.data_model_labels <- dcc_config_react()$schematic$global$data_model_labels
# asset view must be NULL to avoid cross-manifest validation.
# however, it is necessary for filename validation
# always pass asset view, let the data model decide which validations are run
# doing this in a verbose way to avoid warning with ifelse
.asset_view <- NULL
.asset_view <- selected$master_asset_view()
if (!is.null(dcc_config_react()$schematic$model_validate$enable_cross_manifest_validation) &
isTRUE(dcc_config_react()$schematic$model_validate$enable_cross_manifest_validation)) {
.asset_view <- selected$master_asset_view()
.project_scope <- selected$project()
}

Expand Down

0 comments on commit cde7134

Please sign in to comment.