more permissive checks

R/mod_check_modal.R

@@ -52,13 +52,18 @@ mod_check_modal_server <- function(id, table_data){
 
     # Create output
       if (all(check_result$type %in% c("warning", "success"))) {
-        valid_contributors_table <- TRUE
+        is_valid <- TRUE
         } else {
-          valid_contributors_table <-  FALSE
+          is_valid <-  FALSE
           }
 
     # Pass output
-    return(valid_contributors_table)
+    return(
+      list(
+        is_valid = is_valid,
+        check_result = check_result
+        )
+    )
   })
 }
 

R/mod_read_spreadsheet.R

@@ -103,7 +103,7 @@ mod_read_spreadsheet_server <- function(id) {
     # Hide show spreadsheet on start
     golem::invoke_js("hideid", "show-div")
 
-    # Control show spreadsheet button behaviour based on read
+    # Control show spreadsheet button behavior based on read
     observe({
       if(!is.null(table_data())) {
         golem::invoke_js("reable", "#show_spreadsheet-show_data")
@@ -120,10 +120,14 @@ mod_read_spreadsheet_server <- function(id) {
 
     # General validation of contributors_table ---------------------------
     # Alert modal to check contributors_table validity
+    check_output <- reactive({
+      req(table_data())
+      mod_check_modal_server("check_modal", table_data = table_data())
+      })
+
     is_valid <- reactive({
       if (!is.null(table_data())) {
-        check_output <- mod_check_modal_server("check_modal", table_data = table_data())
-        return(check_output)
+        return(check_output()$is_valid)
         } else {
           return(FALSE)
           }
@@ -144,6 +148,7 @@ mod_read_spreadsheet_server <- function(id) {
       list(
         data = table_data_clean,
         is_valid = is_valid,
+        check_result = reactive(check_output()$check_result),
         upload = reactive(input$upload)
         ))
     })

dev/02_dev.R

@@ -16,6 +16,7 @@ golem::add_module(name = "about_modal")
 golem::add_module(name = "contribs_affiliation_page")
 golem::add_module(name = "check_modal")
 golem::add_module(name = "grant_information")
+golem::add_module(name = "feedback")
 
 ## 2.2 Add dependencies