-
Notifications
You must be signed in to change notification settings - Fork 0
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
v0.1.0 release #94
Merged
Merged
v0.1.0 release #94
Changes from 1 commit
Commits
Show all changes
391 commits
Select commit
Hold shift + click to select a range
d58ce42
fix add_timevars_to_data within merge_meta_with_data
LDSamson 7120125
Fix R CMD check error
LDSamson cdcaaa5
Also remove example from .Rd file
LDSamson 8cdbdba
revert changes for #51
aclark02-arcus 0ffa578
add column specs to internal data
LDSamson f67e271
Renaming the metadata tab `column_specs` to the more fitting name `co…
LDSamson 8da18fe
Perform type conversion with the internally stored column types
LDSamson cc2ab12
use readr instead of vroom for reading csv files, since we are now us…
LDSamson 6e2e6bd
Update renv lockfile, update appdata testthat snapshot
LDSamson a154359
update data_spec vignette
LDSamson 66ab364
Remove line for creating day variable second time.
LDSamson 3d368bf
add synch_time attribute to merged data
LDSamson 23454f1
Remove rows with missing subject_id, if they exist.
LDSamson edc839c
update_review_data: make update_time available as function option, se…
LDSamson 27366fe
Add synch time to merged data and database, part two
LDSamson ff791f7
Update tests
LDSamson 356783f
Update snapshots and documentation
LDSamson 0350241
Fix app feature 4
LDSamson 2d71804
Update version and news, update test results
LDSamson a8b2fd6
Merge pull request #47 from openpharma/ls-045-restructure-data-import
jthompson-arcus 8972a69
Merge branch 'dev' into ls-046-improve-synch-time-notification
LDSamson 844e839
ensure sycnh_time attribute is created in merged data even if it is m…
LDSamson 2a5cbdf
Add synch_time to raw data, and clearly describe the default.
LDSamson ea7c696
keep db_update time the same time as the synch_time in study_data, ev…
LDSamson 58e1da4
Merge pull request #60 from openpharma/ls-046-improve-synch-time-noti…
jthompson-arcus 2ad41dd
Add user roles to config in a more machine-readable format
LDSamson 803426b
Ensure only the valid and available roles are passed onto the applica…
LDSamson 9607772
review config: Implement that roles can be selected if multiple valid…
LDSamson 63e10fd
review config: add separate helper function to set user role in mod_r…
LDSamson 626a084
review_config: Add test for feature of changing user role in mod_revi…
LDSamson 5edc33c
Update version
LDSamson a787d6e
Improve sanitizing role input. Rename app role helper functions (to a…
LDSamson 500c9cb
Fix all_roles argument in get_valid_roles, for consistency within the…
LDSamson 7ea477c
Allow list-type input format for user roles roles.
LDSamson a9d9f82
merge conflict resolution, bumping dev version
aclark02-arcus 2633316
Add purrr to suggests
aclark02-arcus 00420f0
remove duplicate na.omit() calls and make it safer
aclark02-arcus fdd2ad6
Check for empty `completed_events` in `fig_timeline()`
jthompson-arcus f7996f6
Update version and NEWS
jthompson-arcus a5a986c
Correct version number
jthompson-arcus 9f96c2a
Merge pull request #66 from openpharma/jt-fix_fig_timeline_bug
LDSamson ec33daf
Merge branch 'dev' into ls-061-make_roles_available
LDSamson 70d4bee
Remove reactive() in reactiveValues() complexity. Fix tests
LDSamson 980a7d5
Increment version number
jthompson-arcus 0c918e0
Merge pull request #63 from openpharma/ls-061-make_roles_available
jthompson-arcus 1f10fa4
query_add: Add user role when adding new query; remove redundant quer…
LDSamson f5ed728
update metdata.rda
aclark02-arcus a1b3e3a
update dev version
aclark02-arcus 7761131
query_add: ensure that user without role cannot add a query. Add test…
LDSamson c0befec
query_follow_up: implement reviewer role in mod_query_follow_up
LDSamson 03af5d5
query_follow_up: ensure a follow-up query can only be written if ther…
LDSamson 9d4d2f7
Update internal datasets
LDSamson f8859d3
Update .png snapshots with correct fixed synch date in feature 1 test
LDSamson 4ad386d
Fix some small minor errors (docs update, snapshot update)
LDSamson b621e3d
Update version and news
LDSamson f2253f2
update news
aclark02-arcus 0eec2fd
Merge pull request #57 from openpharma/ac-50-51
LDSamson ffda246
Merge branch 'dev' into ls-061-add-roles-to-queries
LDSamson 8058621
Increment version
LDSamson a2b09b0
Add roles to mod_report
LDSamson ba3f850
Add roles to review_forms
LDSamson f428123
Restrict roles allowed to review to the ones mentioned in the config …
LDSamson 5c00d20
Update version and news
LDSamson 5783594
Merge pull request #64 from openpharma/ls-061-add-roles-to-queries
jthompson-arcus 4cd6562
Add more explicit warning if no user role or user name is available
LDSamson db0260a
make logical test slightly safer
LDSamson e552fac
Ensure app can handle an empty roles variable
LDSamson 0efb311
Fix pkgdown yml file
LDSamson 2e0e30e
Fix unintended randomness in test outcome
LDSamson 42a328a
Merge branch 'dev' into ls-061-add-roles-to-form-review-and-report
LDSamson 39ebcb7
Revert "Fix pkgdown yml file"
LDSamson 26cfa5e
Merge branch 'dev' into ac-58
aclark02-arcus a138800
move pkgload to 'Suggests' and then revert changes to production/renv…
aclark02-arcus 224ec6b
change renv profile back to dev
aclark02-arcus 9d2b7ee
Add code that helps users update the manifest file for deployment to …
aclark02-arcus 690744d
Add manifest.json file to .Rbuildignore
aclark02-arcus 138c419
Add small description/purpose of the app.R file
LDSamson 9e809f7
Changing renv profile names to 'full' and 'minimal'
LDSamson 2b0109e
Increment package version
LDSamson bcd7831
Merge pull request #59 from openpharma/ac-58
LDSamson 5161fb1
Merge branch 'dev' into ls-061-add-roles-to-form-review-and-report
LDSamson 12f82bc
Increment version
LDSamson 5734e0b
Fix bug in output of get_valid_roles in case no valid role is available
LDSamson c5aaa44
ignore .rds files so that they are not accidentily shared in remote repo
aclark02-arcus 5fca650
add .rscignore for posit connect deployers
aclark02-arcus 80582f0
Increment version number to 0.0.0.9015
aclark02-arcus f3bfa6f
update news and version num
aclark02-arcus be68388
Merge pull request #65 from openpharma/ls-061-add-roles-to-form-revie…
jthompson-arcus 4e97fdd
Merge branch 'dev' into ac-tiny
LDSamson b073e42
add .rscignore to .Rbuildignore
LDSamson 4a50593
Increment version
LDSamson 47e993a
Merge pull request #68 from openpharma/ac-tiny
LDSamson f98dd0b
Remove calls to the clinsight package to prevent an error when using …
LDSamson c7658f4
Update news
LDSamson 0b6acb4
Increment version
LDSamson 25012e5
Revert "Remove calls to the clinsight package to prevent an error whe…
LDSamson 2999bfe
Fix bug by using the golem::pkg_path instead of the yaml for retrievi…
LDSamson 88faf5b
Remove rogue comma
aclark02-arcus 2345b75
Merge pull request #70 from openpharma/ls-69-bug-fix-run_dev
jthompson-arcus 5d061e3
Add form-level default values and required metadata columns to intern…
LDSamson b0e1efc
Add get_form_level_data() to clean form-level data, including several…
LDSamson 0e3777d
Update get_metadata() function to use get_form_level_data(), add some…
LDSamson 48558c5
Update metadata.xlsx, renaming the groups tab to form_level_data, add…
LDSamson a63f0e3
Add form_level data in app_vars by adding it to get_meta_vars
LDSamson 7956bd4
Ensure mod_study_forms uses the new location of form_level_data. Rest…
LDSamson ef6a2d8
Disable review options if review is not required
LDSamson 78eadaf
Remove forms of which review is not required from the summary data se…
LDSamson 2e4c6c7
Increment version
LDSamson b062b69
Update news
LDSamson 67a1840
Fix metadata documentation
LDSamson d3c898e
Adjust `datatable_custom()` to include "Scroller" extension and "scro…
jthompson-arcus dd64ab9
Move caption/title to table wrapper
jthompson-arcus 73cc3e7
Update snaps for 'app-feature-1'
jthompson-arcus b30e9c1
Update snaps for 'app-feature-2'
jthompson-arcus ac06e0a
Update snaps for 'app-feature-3'
jthompson-arcus b2f0079
Update snaps for 'mod_navigate_review'
jthompson-arcus 9aa427e
Update snaps for 'mod_study_forms'
jthompson-arcus 7e98d8d
Update `datatable_custom()` test
jthompson-arcus 3773026
Update version and NEWS
jthompson-arcus d3baeba
Oops
jthompson-arcus c7d55c0
Fix roles are not read correctly when using shinymanager
LDSamson 05eb57f
Fix error if an input is unavailable when using shinymanager
LDSamson 507bb75
Test that no user name or user role error occurs with shinymanager
LDSamson 3eacd33
Update news
LDSamson 0b98c19
Update version
LDSamson 2fa6356
Handle cases where code is inserted in table title
jthompson-arcus f4b88dd
Update R version and packages
LDSamson 338f6d9
Fix golem_config
LDSamson 42d3d05
disable 'compact' view for bslib::value_box() (see https://github.com…
LDSamson f987497
set bslib requirement to 0.6.2 to ensure we can set bslib-value-box-h…
LDSamson f82d891
tweak header widgets appearance a bit
LDSamson 18a64b1
Use theme instead of deprecated argument theme_color
LDSamson 87d3acd
Update app feature 1 test
LDSamson 3a094b6
Update app feature 2 test
LDSamson 4d187e7
Update app feature 3 test
LDSamson 9a8c310
Update app feature 4 snapshots
LDSamson 73457e3
Update mod_navigate_participants test snapshots
LDSamson 59311e3
Update mod_study_forms, mod_nav_review, and mod_main_sidebar tests
LDSamson aba878a
Update authentication test snapshots
LDSamson ee65d55
Update R version in GHA yaml
LDSamson 3d125dd
add libcurl system package manually to GHA
LDSamson 3bba8fc
Prevent new ggplot2 warning by using geom_annotate in compact timeline
LDSamson 3336342
Move shinymanager to suggest
LDSamson 7ddb2c0
Ensure it is installed in the 'full' but not in the 'minimal' profile
LDSamson 2acd34c
Create helpful prompts to install shinymanager if needed
LDSamson de0d78f
Skip shinymanager-related tests if shinymanager is not installed
LDSamson fd712a7
Update test descriptions of shinymanager function tests
LDSamson a69626a
Update news
LDSamson 36d16d1
Increment version
LDSamson 3c46cfb
Respect function arg `selection`
jthompson-arcus b5ab475
Merge pull request #78 from openpharma/ls_77_bug_fix_shinymanager
jthompson-arcus 684af10
Merge branch 'dev' into ls-81-move-shinymanager-to-suggests
LDSamson bab150c
Merge branch 'dev' into jt-74-improve_datatables
jthompson-arcus 9f7aaf0
Restructure `datatable_custom()`
jthompson-arcus 771b6d9
Update documentation
jthompson-arcus 14e9b67
Import `modifyList()`
jthompson-arcus b26335b
Update selected query dataTable
jthompson-arcus e515381
Update snaps
jthompson-arcus 0eb51f8
Small documentation update
jthompson-arcus 63e1d02
Tweak css for header widgets a bit more
LDSamson 0b14be3
Update to snapshot of 12 September 2024
LDSamson 0eac1f2
Reflow code for readability
jthompson-arcus c78fdb1
Fix issue when title is `NULL`
jthompson-arcus 12697d6
Update snaps
jthompson-arcus af1dec5
Update version number
jthompson-arcus 1a3c060
Update version number in config...
jthompson-arcus 26aed39
Merge pull request #75 from openpharma/jt-74-improve_datatables
LDSamson 0eeb341
Merge branch 'dev' into ls-81-move-shinymanager-to-suggests
LDSamson a96a257
Generalize shinymanager login screen; use a non-existent mail address…
LDSamson 89dd054
Increment version
LDSamson 88e0301
Merge pull request #82 from openpharma/ls-81-move-shinymanager-to-sug…
jthompson-arcus 8968b50
Merge branch 'dev' into ls-043-add-review-required-settings
LDSamson a0db067
Merge branch 'dev' into ls-29-update-R-and-packages
LDSamson c8a4cf9
Update snaps
LDSamson 42ad0ec
Increment version
LDSamson 488192f
Update contributing authors
jthompson-arcus 7d85a69
Increment version number
jthompson-arcus ef5da53
add aaron's orcid to `DESCRIPTION`
aclark02-arcus 5a649de
Update documentation
jthompson-arcus ea84049
Add ORCID from Leon
LDSamson 6a9704e
Merge pull request #85 from openpharma/jt-update_contributors
LDSamson 9aff424
Merge branch 'dev' into ls-29-update-R-and-packages
LDSamson e8fd4e6
Increment version
LDSamson 02b41fe
Merge branch 'dev' into ls-043-add-review-required-settings
LDSamson 9b6d166
Increment version
LDSamson 63e22fe
Merge pull request #73 from openpharma/ls-043-add-review-required-set…
LDSamson 563a5ad
Merge branch 'dev' into ls-29-update-R-and-packages
LDSamson 35ae4c5
Increment version
LDSamson c4bb835
Use latest snapshot
LDSamson 5478e67
Merge pull request #79 from openpharma/ls-29-update-R-and-packages
LDSamson 50769c7
Improve mod_study_form ui layout
LDSamson f8f0efd
Use sidebar layout for mod_study_forms
LDSamson b78dfb4
Use sidebar layout for mod_common_forms
LDSamson 8a33692
Improve figure legend - larger, and hidden with popover
LDSamson f3944a8
Align info text
LDSamson a38185a
remove css overflow.x=scroll for basic timeline (not needed with late…
LDSamson 471e742
CSS: use minimum text size in app value boxes
LDSamson 7939947
Slightly bigger figs if only one or two are shown
LDSamson 7ec879e
Update snaps
LDSamson 1d43648
link to homepage in logo
LDSamson 5184bf8
Increment version
LDSamson d24fa2a
Merge pull request #87 from openpharma/ls-improve-UI
LDSamson 854564b
Add form_repeat to common_forms table
LDSamson 5ab8ee0
Change metadata table_names: form_repeat will now show as 'N' in a cu…
LDSamson 7b4d166
Remove rownames from common_forms, since it is a bit redundant now we…
LDSamson 26db26a
Add form_repeat number to the selected query item name, but only if t…
LDSamson 1accb96
Improve visibility in tables with long query texts
LDSamson 449b8d4
Update snaps
LDSamson a250a9c
Ensure query type shows up in query details
LDSamson 8716a00
Add additional integration tests for viewing queries, responding to q…
LDSamson 7c0a80b
Update news
LDSamson 9d5080d
Increment version
LDSamson 01063ce
Fix some issues with mod_report
LDSamson e133ad8
Ensure query type shows up in reports
LDSamson 052bff0
Adjust test for mod_report
LDSamson 1ac7f3f
Update news
LDSamson 0e8cb22
Increment version
LDSamson 6b5afe6
Remove flaky input values from new integration test
LDSamson 1e58c61
of course that change also needs new snapshot files..
LDSamson 3c29424
add some additional waiting time in new test
LDSamson 4802815
Ensure break will not occur at a whitespace
LDSamson 9b86234
Remove some unneeded default settings overrides in query tables
LDSamson e93fe08
Improve query table in mod_navigate_review (now also show query type …
LDSamson 701a752
Update snaps
LDSamson 308f101
Update some older .png shinytest2 snapshot so that the latest UI is s…
LDSamson 933a000
Ensure that form_repeat_name displayed in duplicate query names can b…
LDSamson 857d864
Add global vars to resolve some R package notes
LDSamson 61c30b8
Update docs (resolves R CMD warning)
LDSamson 207db4c
Merge pull request #92 from openpharma/ls-fix-mod_query_issues
jthompson-arcus fe74719
Merge branch 'dev' into ls-fix-reports
LDSamson 8b79c33
Increment version
LDSamson 145f89c
Show user role in prepare report modal in mod_report
LDSamson b01b565
Add additional test for downloading PDF report
LDSamson 29a2aa5
Ensure report type is included in test of mod_report, test for creati…
LDSamson 9b6e76e
Merge pull request #93 from openpharma/ls-fix-reports
LDSamson f7b6360
Increment version number to 0.1.0
LDSamson bce6538
don't use full_width=T for PDF since it relies on a deprecated latex …
LDSamson 88cffc0
Delete unused old file
LDSamson 4bfe038
Update test descriptions
LDSamson 528e51b
Sort start page based on review status
LDSamson d41a50d
Update Snapshot headers after updatinng test descriptions
LDSamson 329daea
Do not upload snapshots automatically with GHA; GHA fails if we reach…
LDSamson 76b0c77
Fix that selected regions were not preserved after changing them and …
LDSamson 5697a60
Add short feature description to all feature tests
LDSamson 12cf77b
Update snapshots
LDSamson 3e37045
Ensure that creating PDF reports does not error with long folder name…
LDSamson e5dcc31
Give shinytest2 some more time to think
LDSamson 60f9a2e
Improve traceability matrix in report
LDSamson 274a233
trim down validation report to only test report with traceabilty matrix
LDSamson 9428066
Update test results
LDSamson File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jthompson-arcus I suspect that this change could solve your problem of not being able to render PDF reports under Windows, closely related to this bug I reported previously: rstudio/rmarkdown#2556