Skip to content

Commit 3171906

Browse files
committed
fix: Type of building now handled as 'not mandatory' but keeps the XLSX data range
1 parent aca7359 commit 3171906

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

NEWS.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@
33

44
* `annex_validate()` will no longer throw an error for variables which have
55
no lower/upper bound set (`Other` and `PMOther`)
6-
* The XLSX template now has a dropdown and data validation for sheet META-Home,
7-
column 'Type of building'. `annex_validate()` now checks for empty cells
8-
and throws a warning (as for other columns, e.g., 'Ventilation type').
6+
* The XLSX template now has a dropdown (pre-defined set of values; data range)
7+
for sheet META-Home column 'Type of building'. Not mandatory, no special
8+
check by `annex_validate()`.
99

1010
# annex 0.2-12
1111

R/validate.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -625,7 +625,7 @@ get_required_columns <- function(sheet) {
625625

626626
# Definition
627627
def <- list("META-Study" = c("Contact", "Institution"),
628-
"META-Home" = c("Location: Country", "Ventilation type", "Type of building"),
628+
"META-Home" = c("Location: Country", "Ventilation type"),
629629
"META-Variable" = "Variable unit")
630630

631631
return(if (sheet %in% names(def)) def[[sheet]] else NULL)

inst/template/template.xlsx

-3 Bytes
Binary file not shown.

0 commit comments

Comments
 (0)