Skip to content
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.

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

Use parameters in get_hipaa_disclosure_log_from_ehr_fhir_logs.R #162

Merged

Conversation

pbchase
Copy link
Contributor

@pbchase pbchase commented Jun 25, 2024

@saipavan10-git, please add better test data and tests

@pbchase pbchase requested a review from saipavan10-git June 25, 2024 19:43
@pbchase
Copy link
Contributor Author

pbchase commented Jun 25, 2024

Consider reading the FHIR logs like this:

redcap_ehr_fhir_logs <- dplyr::tbl(conn, "redcap_ehr_fhir_logs") |>
  dplyr::filter(.data$resource_type == "Patient" &
                  .data$mrn != "") |>
  head(n = 30) |>
  dplyr::collect()

Then mutate ehr_id and created_at to the mix of values you need for testing the parameters

saipavan10-git and others added 4 commits June 30, 2024 21:51
Add additional tests to for the filters in  get_hipaa_disclosure_log_from_ehr_fhir_logs() function.
This allows us to use the mutated data frame as a virtual table (view) in a DuckDB connection without overwriting the data.
@pbchase pbchase merged commit 5590ed4 into develop Dec 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants