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

Devex: Resolve merge conflicts dev #143

Merged
merged 88 commits into from
Nov 20, 2024
Merged
Changes from 1 commit
Commits
Show all changes
88 commits
Select commit Hold shift + click to select a range
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
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
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
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
f299d4a
Resolve merge conflicts with `dev`
jthompson-arcus Nov 4, 2024
63dc37d
Resolve merge conflicts with `dev`
jthompson-arcus Nov 5, 2024
0e6b4af
Update version number
jthompson-arcus 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
dfdf945
Decode shinyproxy user name with base64 encoding, and return a warnin…
LDSamson Nov 1, 2024
dda0b05
Update news and version
LDSamson Nov 4, 2024
d56c4b1
Add a check if the package base64enc is installed within the `decode_…
LDSamson Nov 4, 2024
bab7a92
Update test database so a form with different review status per item …
LDSamson Nov 7, 2024
a37ff1d
Update mod_review_forms logic to ensure review was saved successfully
LDSamson Nov 7, 2024
7ac3278
Update mod_review_forms tests
LDSamson Nov 13, 2024
29eb7a0
Add all common vars to rows_update, so that review status is only upd…
LDSamson Nov 13, 2024
9940944
Update news and description
LDSamson Nov 15, 2024
bafae70
Resolve R CMD documentation note
LDSamson Nov 15, 2024
d21f519
Mark functions as internal for package website
LDSamson Nov 15, 2024
7efa574
Increase wait time to pass tests within Alpine Docker
LDSamson Nov 15, 2024
9d8555b
Update Alpine Docker test results
LDSamson Nov 15, 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
5d52f83
Add pkgdown gha workflow
aclark02-arcus Oct 1, 2024
bbf2b53
don't rely on renv.lock
aclark02-arcus Oct 1, 2024
e9ed1a4
update news
aclark02-arcus Oct 1, 2024
734f9ca
Update pkgdown workflow for use on dev and devex
LDSamson Nov 19, 2024
92677b0
Small NEWS.md wording update
jthompson-arcus Nov 19, 2024
573c633
Merge pull request #138 from openpharma/ls_create_v0.1.1
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
993035f
Resolve merge conflicts with `dev`
jthompson-arcus Nov 20, 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
6 changes: 4 additions & 2 deletions R/fct_SQLite.R
Original file line number Diff line number Diff line change
@@ -122,17 +122,19 @@ db_add_primary_key <- function(con, name, value) {
db_add_log <- function(con) {
DBI::dbCreateTable(con, "all_review_data_log",
c(id = "INTEGER PRIMARY KEY AUTOINCREMENT", review_id = "INTEGER NOT NULL",
reviewed = "CHAR", comment = "CHAR", reviewer = "CHAR", timestamp = "CHAR", status = "CHAR",
edit_date_time = "CHAR", reviewed = "CHAR", comment = "CHAR",
reviewer = "CHAR", timestamp = "CHAR", status = "CHAR",
dml_type = "CHAR NOT NULL", dml_timestamp = "DATETIME DEFAULT CURRENT_TIMESTAMP"))
DBI::dbExecute(con, paste(
"CREATE TRIGGER all_review_data_update_log_trigger",
"AFTER UPDATE ON all_review_data FOR EACH ROW",
"BEGIN",
"INSERT INTO all_review_data_log (",
"review_id, reviewed, comment, reviewer, timestamp, status, dml_type",
"review_id, edit_date_time, reviewed, comment, reviewer, timestamp, status, dml_type",
")",
"VALUES(",
"NEW.id,",
"OLD.edit_date_time,",
"OLD.reviewed,",
"OLD.comment,",
"OLD.reviewer,",