Skip to content

Export Tabular Results #24

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
the-mayer merged 26 commits into from
Dec 4, 2025
Merged

Export Tabular Results #24

the-mayer merged 26 commits into from
Dec 4, 2025

Conversation

@the-mayer
Copy link
Collaborator

@the-mayer the-mayer commented Dec 4, 2025

Overview

This PR partially addresses #23 by adding a tabular export for basic CFDE metrics. Results are compiled into a multi-tabbed Excel workbook and optionally into individual CSV files. Includes tests and documentation for:

  • export_tabular() -assembles analytics outputs into a structured, date-stamped tabular format
  • get_ga_basic() - retrieves basic Google Analytics metrics filtered by NIH Core Project Number

Next Steps

May need some furhter discussion oto determine the scope of the Summary sheet:

  • Should the summary reflect a concise roll-up of metrics from all tabs?
  • Or should it remain focused on a single tab, eg Google Analytics?
  • Which metrics will be most valuable to end users

Feedback welcome!

@the-mayer
prep
96a3f6e
@the-mayer
beginnings of export function
6edb88a
@the-mayer
sp
64288b7
@the-mayer
prep for google analytics inclusion
88a157f
@the-mayer
ga_meta by propertyId helper
2b86a38
@the-mayer
misc updates
fb2348f 
- NAMESPACE
- docs
- gitignore
@the-mayer
updates
7680936 
- add missing namespace deps
- update docs
- refactor auth
@the-mayer
rename
1e06f2d
@the-mayer
update param, disambiguate from github tokens
456e1eb
@the-mayer
make case insensitive
0cddefd
@the-mayer
cleanup
15675ca
@the-mayer
utilize file param for workbook export
436868d
@the-mayer
add messaging, if empty return
e74a428
@the-mayer
fix regex filter, when multiple core projects are specified
8a65736
@the-mayer
add test for get_ga_basic()
761bbf5
@the-mayer
update docs
8d05464
@the-mayer
basic validation (ensure project numbers are unique)
d3a4cf7
@the-mayer
add openxlsx to DESCRIPTION
f0f9851
@the-mayer
misc R CMD check fixes
2c28587
@the-mayer
update auth progression
847d560
@the-mayer
additional description about auth
0d5fd5b
@the-mayer
add test for export excel
010157c
@the-mayer
update param, and test
939022c
@the-mayer
tidyselect fix
a95c835
@the-mayer
sync with main
c790023
@the-mayer
add optional csv export for tabs, change function name to reflect
cd5b128
@the-mayer the-mayer self-assigned this Dec 4, 2025
@the-mayer the-mayer linked an issue Dec 4, 2025 that may be closed by this pull request
Bespoke function to create excel spreadsheet of results #23
Closed
5 tasks
@the-mayer the-mayer requested a review from seandavi December 4, 2025 15:24
seandavi
seandavi approved these changes Dec 4, 2025
View reviewed changes
Copy link
Member

@seandavi seandavi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR establishes a great baseline for exporting tabular results.

I'm in favor of merging and then iterating in separate issues to address stakeholder needs and extend documentation and functionality.

@the-mayer the-mayer merged commit 129db0e into main Dec 4, 2025
1 check passed
@the-mayer the-mayer deleted the 23-bespoke-function-to-create-excel-spreadsheet-of-results branch December 4, 2025 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seandavi seandavi seandavi approved these changes

Assignees

@the-mayer the-mayer

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Bespoke function to create excel spreadsheet of results

3 participants

@the-mayer @seandavi