Merge pull request #60 from aghaynes/main

bug fix
CTU-Bern · Oct 9, 2024 · 5523235 · 5523235
2 parents 035464b + 0220da9
commit 5523235
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,7 +1,7 @@
 Package: CTUCosting
 Type: Package
 Title: Machinery for CTU costings
-Version: 0.6.0
+Version: 0.6.1
 Authors@R:
     c(
     person(given = "Alan G.", family = "Haynes", role = "cre",
@@ -32,7 +32,7 @@ Imports:
     tidyr,
     vctrs,
     writexl
-RoxygenNote: 7.2.3
+RoxygenNote: 7.3.0
 Suggests:
     bsicons,
     bslib,

diff --git a/NEWS.md b/NEWS.md
@@ -1,3 +1,7 @@
+# CTUCosting 0.6.1 (2024-10-09)
+
+* bug fix: notes were stopping the PDF from being generated
+
 # CTUCosting 0.6.0 (2024-09-27)
 
 * restyle to approximate DCR letterhead, including DCR logo

diff --git a/R/get_notes.R b/R/get_notes.R
@@ -55,7 +55,7 @@ notes_filter <- function(notes, selected_workpackages, fte){
       unique() |>
       na.omit()
     print(paste("NOTESFILTER: ", services))
-    if(fte) services <- c(services, "full_time_equivalent")
+    if(fte & "full_time_equivalent" %in% names(notes)) services <- c(services, "full_time_equivalent")
     out <- notes[services]
     print("filtered notes names")
     print(names(out))