Skip to content

Commit bfd0385

Browse files
Improves on error message on loading reporter (#374)
# Pull Request <!--- Replace `#nnn` with your issue link for reference. --> - Fixes #365 ### Changes description - Improves error message as users may have renamed the file <img width="993" height="273" alt="2025-08-07_15-22" src="https://github.com/user-attachments/assets/f2541180-27ea-439e-941d-165c7dafda67" /> ### Alternative: - Remove naming pattern enforcement _(making sure it's a zip file)_ - @Polkas / @gogonzo do you have insights why this is so strict? (source PR: #251) https://github.com/insightsengineering/teal.reporter/blob/f776a9e217141b561b1537ca076115d2147d49ea/R/LoadReporterModule.R#L97 --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
1 parent fb8fb4a commit bfd0385

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

R/LoadReporterModule.R

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,12 @@ load_json_report <- function(reporter, zip_path, filename) {
134134
}
135135
)
136136
} else {
137-
shiny::showNotification("Failed to load the Reporter file.", type = "error")
137+
shiny::showNotification(
138+
paste(
139+
"Failed to load the Reporter file.",
140+
"Please make sure that the filename starts with `report_`."
141+
),
142+
type = "error"
143+
)
138144
}
139145
}

0 commit comments

Comments
 (0)