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

Diff: dev vs main #140

Draft
wants to merge 219 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
219 commits
Select commit Hold shift + click to select a range
92d4659
Increment version number to 0.1.0.9000
LDSamson Sep 30, 2024
d62de80
Add pkgdown gha workflow
aclark02-arcus Oct 1, 2024
3381f26
don't rely on renv.lock
aclark02-arcus Oct 1, 2024
78e9927
update certain functions with '@keywords internal' for pkddown
aclark02-arcus Oct 1, 2024
19b8f83
Increment version number to 0.1.0.9000
aclark02-arcus Oct 1, 2024
279bc5c
update news
aclark02-arcus Oct 1, 2024
9c03622
Merge branch 'dev' into ac-95
aclark02-arcus Oct 1, 2024
0aff3a9
Update golem config with new app version
aclark02-arcus Oct 1, 2024
0f08edf
Merge branch 'ac-95' of https://github.com/openpharma/clinsight into …
aclark02-arcus Oct 1, 2024
f927749
Add callback to "go to form" on double click event
jthompson-arcus Oct 8, 2024
a7b8eaa
Pass additional modifying functions in arguments of `merge_meta_with_…
jthompson-arcus Oct 9, 2024
f091000
Recode to use base R and character values
jthompson-arcus Oct 9, 2024
1ff5c2d
Move additional merging functions to metadata
jthompson-arcus Oct 9, 2024
5d4a444
Adjust `%||%` to check for empty values, not just `NULL`
jthompson-arcus Oct 10, 2024
6451405
Specify `add_fns` as `post_merge_fns`
jthompson-arcus Oct 10, 2024
2270fdc
Add `pre_merge_fns` setting
jthompson-arcus Oct 10, 2024
7cf9ee4
Add inline function to check existence of an object for `dplyr` mutat…
jthompson-arcus Oct 10, 2024
ba6c086
Impute `event_name` from `event` instead of `event_name`
jthompson-arcus Oct 10, 2024
8dc00e0
Fix broken tests
jthompson-arcus Oct 10, 2024
8154349
Further generalize `add_timevars_to_data()`
jthompson-arcus Oct 10, 2024
325c27d
Initialize region field
jthompson-arcus Oct 10, 2024
2506500
Add mid-merging functions
jthompson-arcus Oct 11, 2024
c9755b7
Update version and NEWS
jthompson-arcus Oct 11, 2024
ed64d65
Move logic to `dblclick_to_form()` function
jthompson-arcus Oct 11, 2024
f48ebac
Remove duplicate type conversion
jthompson-arcus Oct 14, 2024
6087b0e
Create `apply_custom_functions()` to handle
jthompson-arcus Oct 14, 2024
a980920
Clean up metadata settings element generation
jthompson-arcus Oct 14, 2024
c90e778
Remove extraneous pipe
jthompson-arcus Oct 14, 2024
4d7fd45
Reset `%||%` to standard evaluation
jthompson-arcus Oct 14, 2024
6adecea
Display message if using user supplied data.
jthompson-arcus Oct 14, 2024
b001c9b
Remove `event_name` from required columns
jthompson-arcus Oct 14, 2024
568a05f
Resolve test errors
jthompson-arcus Oct 14, 2024
12b28f1
Specify the `parent.frame()` as the environment to check existence wi…
jthompson-arcus Oct 15, 2024
ea4714a
Add tests for `%|_|%`
jthompson-arcus Oct 15, 2024
3e9dc7b
Change `warning()` to `cat()`
jthompson-arcus Oct 15, 2024
a4d88ba
Merge pull request #104 from openpharma/jt-generalize_meta_merging
LDSamson Oct 15, 2024
e5e4da4
Add primary key field to tables
jthompson-arcus Oct 16, 2024
1cffcdf
Temporary fix to preserve current functionality
jthompson-arcus Oct 16, 2024
ec0c753
Re-create `review_testdb.sqlite`
jthompson-arcus Oct 16, 2024
193f05d
Repair `mod_review_form` tests
jthompson-arcus Oct 16, 2024
58435dc
Repair `fct_SQLite` tests
jthompson-arcus Oct 16, 2024
8f59b04
Rebuild `testdb.sqlite`
jthompson-arcus Oct 16, 2024
ec868d8
Temporary fix for adding queries to work as expected
jthompson-arcus Oct 16, 2024
804f7de
Add preliminary logging table
jthompson-arcus Oct 16, 2024
48314ae
Add "all_review_data_log" to list of expected tables
jthompson-arcus Oct 16, 2024
53bb43b
Merge branch 'dev' into jt-dblclick_overview_tbl
jthompson-arcus Oct 17, 2024
4201ebe
Add `dblclick` callback to other appropriate tables
jthompson-arcus Oct 17, 2024
14a96fa
Remove unnecessary `session$`
jthompson-arcus Oct 17, 2024
4f78d7b
Repair `mod_navigate_review` tests
jthompson-arcus Oct 17, 2024
0c5cbe0
Repair app feature 3 tests
jthompson-arcus Oct 17, 2024
d4d041d
Repair app feature 2 tests
jthompson-arcus Oct 17, 2024
85b492e
Repair app feature 1 tests
jthompson-arcus Oct 17, 2024
9ce1c9b
Add help files
jthompson-arcus Oct 17, 2024
60b748a
Make sure expression is appropriately defined for `%|_|%` in `add_tim…
jthompson-arcus Oct 17, 2024
f35e21d
Update version
jthompson-arcus Oct 17, 2024
5b4faed
Further generalization
jthompson-arcus Oct 18, 2024
d5d1d52
Repair `fct_appdata` tests
jthompson-arcus Oct 18, 2024
6aa89a3
Repair custom metadata test
jthompson-arcus Oct 18, 2024
abdcde2
Update documentation
jthompson-arcus Oct 18, 2024
9eb651a
Update global variables
jthompson-arcus Oct 18, 2024
2cc5bcf
Resolve issue with unit derivation
jthompson-arcus Oct 22, 2024
6461997
Add `form_type` class to app data
jthompson-arcus Oct 23, 2024
05b869c
Minimal solution
jthompson-arcus Oct 23, 2024
33742e1
Revert to old code for units
jthompson-arcus Oct 24, 2024
46b0489
Merge pull request #116 from openpharma/jt-gmm_follow_up
LDSamson Oct 24, 2024
655662b
Merge branch 'dev' into jt-107-extend_create_table
jthompson-arcus Oct 24, 2024
9a616fb
Merge branch 'dev' into jt-dblclick_overview_tbl
jthompson-arcus Oct 24, 2024
a84a230
Incorporate "dblclick" to app feature 1 test
jthompson-arcus Oct 24, 2024
b30627c
Update version and NEWS
jthompson-arcus Oct 24, 2024
1d5bdd5
Update feature 01 test description, add back test of navigating by cl…
LDSamson Oct 25, 2024
278b99c
Improve news description
LDSamson Oct 28, 2024
24d8ea9
Merge pull request #103 from openpharma/jt-dblclick_overview_tbl
LDSamson Oct 28, 2024
492945b
Fix warnings in `apply_edc_specific_changes`
LDSamson Oct 28, 2024
cafe81d
Update news and version
LDSamson Oct 28, 2024
37d46c1
Merge pull request #120 from openpharma/ls_119_fix_warnings
jthompson-arcus Oct 28, 2024
0b2eb31
Update `db_save_review` to update instead of append
jthompson-arcus Oct 28, 2024
37a7c11
Repair tests from `db_save_review()` changes
jthompson-arcus Oct 28, 2024
07e67c3
Merge branch 'dev' into ac-95
aclark02-arcus Oct 28, 2024
6a06ffc
Merge branch 'dev' into jt-107-extend_create_table
jthompson-arcus Oct 28, 2024
3f38993
Simplify `form_types` creation
jthompson-arcus Oct 28, 2024
802bc29
Update version and NEWS
jthompson-arcus Oct 28, 2024
d220c70
Resolve merge conflicts with `dev`
jthompson-arcus Oct 28, 2024
f3a5562
update news and golem-config veresion
aclark02-arcus Oct 29, 2024
10f1c5f
merge conflic on golem_version (again)
aclark02-arcus Oct 29, 2024
08a9aad
update a few functions with '@keywords internal' for pkgdown build
aclark02-arcus Oct 29, 2024
151e465
Merge pull request #102 from openpharma/ac-95
LDSamson Oct 29, 2024
4f9ba5e
Merge branch 'dev' into jt-107-extend_create_table
jthompson-arcus Oct 29, 2024
f884d68
Update version number
jthompson-arcus Oct 29, 2024
91a8c49
Merge branch 'dev' into jt-113-logging_tables
jthompson-arcus Oct 29, 2024
f98f0d2
Only store old records in logging table
jthompson-arcus Oct 29, 2024
056b99e
Add S3 generic to `_pkgdown.yml`
jthompson-arcus Oct 29, 2024
9e55193
Modify update process to only updated/append appropriate records
jthompson-arcus Oct 29, 2024
f66f758
Actually pull the newer record when updated
jthompson-arcus Oct 29, 2024
2ad8cc7
Add `db_version` table to DB
jthompson-arcus Oct 30, 2024
75ad1d1
Add `edit_date_time` to logging table
jthompson-arcus Oct 30, 2024
4c8e2e5
Update all columns not considered in common
jthompson-arcus Oct 30, 2024
3f32584
Correct error in creating primary key tables
jthompson-arcus Oct 30, 2024
b81500d
Update `fct_SQLite.R` tests to reflect changes
jthompson-arcus Oct 30, 2024
d268d14
Update test DB
jthompson-arcus Oct 30, 2024
1cb9afa
Incorporate review logging tests into `test-fct_SQLite.R`
jthompson-arcus Oct 30, 2024
c6dace0
Add logging tests to `test-mod_review_forms.R`
jthompson-arcus Oct 30, 2024
7effc4c
Add index to `all_review_data` table
jthompson-arcus Oct 30, 2024
cd5d630
Utilize UPSERT for DB append/update procedure
jthompson-arcus Oct 31, 2024
564d0f9
Make UPSERT process more robust
jthompson-arcus Oct 31, 2024
af8f76c
Add function `update_db_version()` to migrate old DBs
jthompson-arcus Oct 31, 2024
f2ef3a6
Update version and NEWS
jthompson-arcus Oct 31, 2024
2c633d2
Fix warning message popping up in tests
jthompson-arcus Oct 31, 2024
746033a
Change `create_table.common_events` to `create_table.common_forms`
jthompson-arcus Nov 1, 2024
5121075
Add `form_type` to `metadata$item_expanded`
jthompson-arcus Nov 1, 2024
cf53ca7
Repair `fct_appdata.R` tests
jthompson-arcus Nov 1, 2024
f159dd3
Add `create_table.common_forms()` tests
jthompson-arcus Nov 1, 2024
6779999
Increment version number to 0.1.0.9005
aclark02-arcus Nov 4, 2024
a22a5b5
Decode shinyproxy user name with base64 encoding, and return a warnin…
LDSamson Nov 1, 2024
1e72f44
Update news and version
LDSamson Nov 4, 2024
80534e1
Update workflows
jthompson-arcus Nov 4, 2024
3970db9
Only update website on pushes to `main`
jthompson-arcus Nov 4, 2024
a76fb98
Add `workflow_dispatch` to R CMD check
jthompson-arcus Nov 4, 2024
af8da6a
Merge pull request #127 from openpharma/jt-update_devex_workflows
LDSamson Nov 4, 2024
3b46c22
update 'default_opts' instead and update function documentation
aclark02-arcus Nov 4, 2024
3f377a9
Add a check if the package base64enc is installed within the `decode_…
LDSamson Nov 4, 2024
3290897
Merge pull request #123 from openpharma/ls_use_base64_encoded_usernames
jthompson-arcus Nov 4, 2024
147c521
Simplify form string
jthompson-arcus Nov 4, 2024
d3cf1b0
Update version
jthompson-arcus Nov 4, 2024
3315a24
Resolve merge conflicts with `dev`
jthompson-arcus Nov 4, 2024
f299d4a
Resolve merge conflicts with `dev`
jthompson-arcus Nov 4, 2024
35f49e7
Merge branch 'dev' into ac-colorder
aclark02-arcus Nov 4, 2024
0f03ad3
update test snapshots with colReorder extension
aclark02-arcus Nov 5, 2024
2c53b86
Merge branch 'ac-colorder' of https://github.com/openpharma/clinsight…
aclark02-arcus Nov 5, 2024
a6b7659
add snap from feature 02
aclark02-arcus Nov 5, 2024
4ff0813
update snaps for feature 03
aclark02-arcus Nov 5, 2024
6ce0001
update the rest of the snaps
aclark02-arcus Nov 5, 2024
858bc30
Merge pull request #124 from openpharma/ac-colorder
aclark02-arcus Nov 5, 2024
4b73522
Resolve merge conflicts with `dev`
jthompson-arcus Nov 5, 2024
ce8bb74
Update version number
jthompson-arcus Nov 5, 2024
63dc37d
Resolve merge conflicts with `dev`
jthompson-arcus Nov 5, 2024
0e6b4af
Update version number
jthompson-arcus Nov 5, 2024
02359d4
Merge pull request #118 from openpharma/jt-107-extend_create_table
LDSamson Nov 5, 2024
c9b8cf8
Resolve merge conflicts with `dev`
jthompson-arcus Nov 5, 2024
abe5703
Revert changes to `mod_review_data_fct_helpers.R`
jthompson-arcus Nov 7, 2024
9c4dc72
Update internal documentation
jthompson-arcus Nov 7, 2024
62aa39d
Add index columns and db version to internal data
jthompson-arcus Nov 7, 2024
8e7f752
Create helper function to add new tables
jthompson-arcus Nov 7, 2024
7b7114d
Oopsies
jthompson-arcus Nov 7, 2024
6e65f04
Update UPSERT documentation
jthompson-arcus Nov 7, 2024
4af9905
Ignore `.Renviron` on git
jthompson-arcus Nov 8, 2024
7af507c
Initialize `db_get_table()`
jthompson-arcus Nov 8, 2024
2191a04
Use `db_get_table()` instead of `db_slice_rows()` to retrieve review …
jthompson-arcus Nov 8, 2024
0582b3b
Use `db_get_table()` instead of `db_slice_rows()` in tests
jthompson-arcus Nov 8, 2024
b370ff5
Remove unneeded slicing and filtering
jthompson-arcus Nov 8, 2024
f0837f2
Capture second snapshot
jthompson-arcus Nov 8, 2024
27c9220
Add comment for `idx_cols`
jthompson-arcus Nov 11, 2024
d1c81ea
Use unique names in function arguments from internal package objects
jthompson-arcus Nov 11, 2024
f271502
Merge pull request #115 from openpharma/jt-113-logging_tables
LDSamson Nov 11, 2024
cea4d43
Merge branch 'dev' into jt-113-simplify_review_process
jthompson-arcus Nov 11, 2024
3e1a650
Add trigger for update on id
jthompson-arcus Nov 11, 2024
851a5ba
Check DB version at run time
jthompson-arcus Nov 11, 2024
485f464
Update NEWS and version
jthompson-arcus Nov 11, 2024
7c58504
Migrate to `review_records` from `review_row`
jthompson-arcus Nov 13, 2024
b271add
Update test-fct_SQLite.R
jthompson-arcus Nov 13, 2024
39d8da1
Update test-mod_review_forms.R
jthompson-arcus Nov 13, 2024
43931ed
Update test-mod_main_sidebar.R
jthompson-arcus Nov 13, 2024
d311ab1
Fix R CMD checks
jthompson-arcus Nov 13, 2024
ccd9bb1
Fix `db_get_review()` example
jthompson-arcus Nov 13, 2024
0743880
Helps if you update documentation
jthompson-arcus Nov 13, 2024
e5e8acb
Only UPDATE reviews when review status is changed
jthompson-arcus Nov 14, 2024
0e266d7
Use `DBI` directly instead of `tbl()` and `collect()`
jthompson-arcus Nov 18, 2024
1405702
Fix typo in `user_db` version check
jthompson-arcus Nov 18, 2024
240c4e8
Only update records with new status.
jthompson-arcus Nov 18, 2024
8085287
Remove argument no longer used
jthompson-arcus Nov 18, 2024
a797883
Remove `common_vars` from tests
jthompson-arcus Nov 18, 2024
20f9528
Repair tests with `db_get_table()` returning `data.frame` not `tibble`
jthompson-arcus Nov 18, 2024
6fb21a6
Update `review_testdb.sqlite`
jthompson-arcus Nov 18, 2024
2a7b9e9
Update `mod_review_forms` test with new testing DB
jthompson-arcus Nov 18, 2024
c6ef19c
Update `all_review_data_id_update_trigger` to include `idx_cols`
jthompson-arcus Nov 18, 2024
1a85f40
Add `keys` argument to `db_add_log()`
jthompson-arcus Nov 18, 2024
c427f2d
Resolve some R CMD check issues
jthompson-arcus Nov 18, 2024
2551970
Resolve merge conflicts with `jt-113-simplify_review_process`
jthompson-arcus Nov 19, 2024
d863300
Keep current structure for enabling reviews
jthompson-arcus Nov 19, 2024
0e8b655
Resolve merge conflicts with `main`
jthompson-arcus Nov 19, 2024
a9488cc
small alignment fix
LDSamson Nov 20, 2024
906daf8
Merge pull request #142 from openpharma/jt-resolve_merge_conflicts_main
LDSamson Nov 20, 2024
b8e598e
Merge branch 'dev' into jt-113-simplify_review_process
LDSamson Nov 20, 2024
f57b549
Improve alignment according to https://style.tidyverse.org/syntax.htm…
LDSamson Nov 20, 2024
740a10a
Merge pull request #135 from openpharma/jt-113-simplify_review_process
LDSamson Nov 20, 2024
6ad27a6
Resolve merge conflict with `dev`
jthompson-arcus Nov 20, 2024
a23b462
Generalize `db_get_review()`
jthompson-arcus Nov 20, 2024
6990cf1
Update version and NEWS
jthompson-arcus Nov 20, 2024
3db4274
Update scenario description for two rows
jthompson-arcus Nov 21, 2024
1c675a9
Update language from row to records
jthompson-arcus Nov 21, 2024
c177a30
Grab un-duplicated `rv_records`
jthompson-arcus Nov 21, 2024
d2fa0d6
Remove unnecessary `dplyr::select()`
jthompson-arcus Nov 21, 2024
de24777
Remove unneeded `dplyr::distinct()`
jthompson-arcus Nov 21, 2024
d372afa
Return `db_get_review()` when `...` is empty
jthompson-arcus Nov 21, 2024
8261441
Update to latest test-coverage. Remove package installation since it …
LDSamson Nov 19, 2024
0f4a693
Fix codecov arguments for codecov-actvion V5 https://github.com/marke…
LDSamson Nov 21, 2024
c547862
Print codecov results again
LDSamson Nov 21, 2024
f28f174
Update test-mod_review_forms.R
jthompson-arcus Nov 21, 2024
047b459
Update version and news
LDSamson Nov 21, 2024
7e26f83
Update only one table in `db_save_review()`
jthompson-arcus Nov 21, 2024
ad3f05e
Fix oopsies
jthompson-arcus Nov 21, 2024
b2bd9bb
Merge pull request #139 from openpharma/ls_update_codecov
jthompson-arcus Nov 21, 2024
939566f
Fix merge conflict with `dev`
jthompson-arcus Nov 21, 2024
439d14b
Add test for `db_get_review()` when no filters are specified
jthompson-arcus Nov 22, 2024
0079348
Fix `updated_items_memory`
jthompson-arcus Nov 22, 2024
ca83997
Don't export db_get_review anymore
LDSamson Nov 28, 2024
0f680cd
Update documentation
LDSamson Nov 28, 2024
a08c510
Ensure db_get_review errors before sending a db query if `...` cannot…
LDSamson Nov 28, 2024
f7ba04a
Add additional tests
LDSamson Nov 28, 2024
205896f
Use more consistent naming in the save review process
LDSamson Nov 28, 2024
b8f3f52
Change a dplyr::filter call to base R filter for performance reasons
LDSamson Nov 28, 2024
0154355
Ensure that updated_records_memory has the same columns as in the db
LDSamson Nov 28, 2024
f0144a2
Ensure tests will fail if there is a review discrepancy
LDSamson Nov 28, 2024
5360760
Export review_save_error to ensure errors are captured when saving re…
LDSamson Nov 28, 2024
88f5b04
Update snaps
LDSamson Nov 28, 2024
23b6e91
Also update mod_main_sidebar snaps
LDSamson Nov 28, 2024
6ce5827
Change to base R filter
LDSamson Dec 2, 2024
0e8ecc7
Fix review saving by selecting correct rows to update within the appl…
LDSamson Dec 2, 2024
d8e24d4
Remove the (now) redundant filtering by timestamp for updated_records…
LDSamson Dec 2, 2024
5845568
Merge pull request #137 from openpharma/jt-99-review_by_row
LDSamson Dec 2, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions .github/workflows/R-CMD-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,10 @@
# Need help debugging build failures? Start at https://github.com/r-lib/actions#where-to-find-help
on:
push:
branches: [main, dev]
branches: [main, dev, devex]
pull_request:
branches: [main, dev]
branches: [main, dev, devex]
workflow_dispatch:

name: R-CMD-check

Expand Down
34 changes: 20 additions & 14 deletions .github/workflows/test-coverage.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@
# Need help debugging build failures? Start at https://github.com/r-lib/actions#where-to-find-help
on:
push:
branches: [main, dev]
branches: [main, dev, devex]
pull_request:
branches: [main, dev]
branches: [main, dev, devex]
workflow_dispatch:

name: test-coverage
Expand All @@ -15,7 +15,6 @@ jobs:
env:
RENV_PROFILE: "full"
GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
RENV_CONFIG_SANDBOX_ENABLED: false

steps:
Expand All @@ -39,21 +38,28 @@ jobs:

- uses: r-lib/actions/setup-r-dependencies@v2
with:
packages: any::covr
packages: any::covr any::xml2
needs: coverage

- name: Install clinsight
shell: bash
run: R CMD INSTALL --preclean .

- name: Test coverage
run: covr::codecov(
quiet = FALSE,
clean = FALSE,
install_path = file.path(Sys.getenv("RUNNER_TEMP"), "package"),
token = Sys.getenv("CODECOV_TOKEN")
)
run: |
cov <- covr::package_coverage(
quiet = FALSE,
clean = FALSE,
install_path = file.path(normalizePath(Sys.getenv("RUNNER_TEMP"), winslash = "/"), "package")
)
print(cov)
covr::to_cobertura(cov)
shell: Rscript {0}

- uses: codecov/codecov-action@v5
with:
# Fail if error if not on PR, or if on PR and token is given
fail_ci_if_error: ${{ github.event_name != 'pull_request' || secrets.CODECOV_TOKEN }}
files: ./cobertura.xml
plugins: noop
disable_search: true
token: ${{ secrets.CODECOV_TOKEN }}

- name: Show testthat output
if: always()
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
.Rhistory
.RData
.Ruserdata
.Renviron
*.html
*.tmp
~$*
Expand Down
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Package: clinsight
Title: ClinSight
Version: 0.1.1
Version: 0.1.1.9011
Authors@R: c(
person("Leonard Daniël", "Samson", , "lsamson@gcp-service.com", role = c("cre", "aut"),
comment = c(ORCID = "0000-0002-6252-7639")),
Expand Down
2 changes: 1 addition & 1 deletion NAMESPACE
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

S3method(create_table,adverse_events)
S3method(create_table,bm_cytology)
S3method(create_table,common_forms)
S3method(create_table,conc_procedures)
S3method(create_table,continuous)
S3method(create_table,default)
Expand All @@ -26,7 +27,6 @@ export(datatable_custom)
export(date_cols_to_char)
export(db_create)
export(db_get_query)
export(db_get_review)
export(db_save)
export(db_save_review)
export(db_slice_rows)
Expand Down
17 changes: 17 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,20 @@
# clinsight (development version)

## Changed

- Generalized `merge_meta_with_data()` to allow user-defined processing functions.
- Added a feature where, in applicable tables, a user can navigate to a form by double-clicking a table row.
- Fixed warnings in `apply_edc_specific_changes` due to the use of a vector within `dplyr::select`.
- Gave users ability to re-organized the column order in any table.
- Added form type as a class to be used in `create_table()` to display tables.
- Add a logging table to the DB for reviews.
- Simplify pulling data from DB for reviews.
- Review data by records IDs instead of subject & form

## Bug fixes

- The test-coverage GHA workflow is updated so that codecov uploads work again.

# clinsight 0.1.1

## Changed
Expand Down
2 changes: 1 addition & 1 deletion R/app_server.R
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ app_server <- function(
)
# think of using the pool package, but functions such as row_update are not yet supported.
r <- reactiveValues(
review_data = db_slice_rows(user_db, db_table = "all_review_data"),
review_data = db_get_table(user_db, db_table = "all_review_data"),
query_data = collect_query_data(user_db),
filtered_subjects = app_vars$subject_id,
filtered_data = app_data,
Expand Down
Loading
Loading