Commit Graph

2298 Commits

Author SHA1 Message Date
Dan ca5e984915 Merge branch 'dev' of github.com:sartography/cr-connect-workflow into dev 2021-11-11 13:47:43 -05:00
Dan 8cabdd1f96 Bumping to new release of SpiffWorkflow. 2021-11-11 13:47:35 -05:00
Dan Funk 8c166bd029
Merge pull request #423 from sartography/test-protocol-builder-519
Test protocol builder #519
2021-11-11 11:49:57 -05:00
Dan f4f3842059 Fixing a stupid mistake. 2021-11-11 11:45:35 -05:00
Dan 197b75c6bf Quickfix - validation was failing if we had a date object in the task data, but this isn't really a problem. We now use the same JSON serializer as flask does, when doing validations, to better verify behavior with the front end. 2021-11-11 11:18:58 -05:00
mike cullerton c7147274f0 Resolving issues from code review 2021-11-10 17:43:34 -05:00
mike cullerton 0e8c4580ed Moved to instance/config 2021-11-10 17:18:33 -05:00
mike cullerton a8b7cd8bba Merge branch 'dev' into test-protocol-builder-519 2021-11-10 16:32:34 -05:00
Dan Funk 037692f42b
Merge pull request #422 from sartography/chore/group-repeating-validation-536
Fixes #536 - Group + Repeating property validation
2021-11-10 15:28:04 -05:00
Dan Funk e764bf937d
Merge pull request #421 from sartography/bug/validation_of_enum_label_failing_in_sub_workflows
Validation was failing for enum_label() expressions when called withi…
2021-11-10 15:26:03 -05:00
Dan Funk 81f8aed68b
Merge pull request #420 from sartography/drop-tasklog-on-delete
Delete Task log model when deleting a study
2021-11-10 15:20:11 -05:00
Dan Funk 56dc34a209
Merge pull request #414 from sartography/chore/view-as-382
remove email script test line about UTC stuff
2021-11-10 15:18:41 -05:00
Dan Funk 84f364ce37
Merge branch 'dev' into chore/view-as-382 2021-11-10 15:16:20 -05:00
alicia pritchett accdd281fa Fixes #536 - Group + Repeating property validation
For now, we should not allow both group and repeat properties on a field together.
2021-11-09 13:21:44 -05:00
mike cullerton 748c80964f Don't need the `.strip('/')` for the UVA urls 2021-11-09 13:09:01 -05:00
mike cullerton 5cf742d955 remove unused import 2021-11-09 13:07:01 -05:00
mike cullerton d584e735ec Possible way to switch between real and mock PB 2021-11-09 13:03:41 -05:00
Dan d1eae3c15a Validation was failing for enum_label() expressions when called within a sub-process. Possible (but unlikely) that this would occur outside validation. 2021-11-09 12:55:06 -05:00
alicia pritchett cd889c5b67 test that you can't be someone not in ldap 2021-11-09 11:08:55 -05:00
alicia pritchett 024a63b725 new and improved test 2021-11-09 11:05:04 -05:00
alicia pritchett aa1ba59bb1 Delete Task log model when deleting a study
just a minor fix
2021-11-09 10:42:47 -05:00
mike cullerton fe502ad306 Added easy way to switch back and forth between PB urls 2021-11-09 10:29:23 -05:00
mike cullerton b193f56f35 Minor cleanup for readability. 2021-11-08 10:13:07 -05:00
mike cullerton ed5689c1b9 Fixed `test_info_script_documents` to use a mocked call to required_docs.
Fixed `test_get_required_docs` to use new required_docs syntax
study_details now in a list, so we look at the first item in the list
2021-11-08 10:00:19 -05:00
mike cullerton 4e13ea6704 Fixing test mocks
- study_details returned in a list
- added `Q_COMPLETE` and `HSRNUMBER` to user_studies
- `required_docs` totally different from what we had
2021-11-08 09:55:06 -05:00
mike cullerton 6f09b0d7dd Added comment to _is_valid_study.
Fixed get_required_docs. The data returned for get_required_dcs from UVA PB was *very* different than what we had.
2021-11-05 14:37:41 -04:00
mike cullerton d186b24de3 UVA PB returns the info in a list. We want the dictionary in the list 2021-11-05 14:26:21 -04:00
mike cullerton 271d4da5ff Merge branch 'dev' into test-protocol-builder-519 2021-11-05 10:29:20 -04:00
mike cullerton 6d548c2a74 *** WIP ***
Committing to work on another ticket
There are TODOs in study_service
2021-11-05 09:59:14 -04:00
Dan 9376d3deaf Don't error out trying to send an error about invalid review types. 2021-11-04 13:32:57 -04:00
Dan Funk 3a36a48b88
Merge pull request #418 from sartography/convert-timestamp-499
Added ordered arguments to the get_localtime script #499
2021-11-03 14:28:57 -04:00
Dan Funk aeaacaf21f
Merge pull request #417 from sartography/email-data-script-487
get_email_data bug #487
2021-11-03 14:27:30 -04:00
mike cullerton b03e596dcf One thing at a time; remove HSRNUMBER for now 2021-11-03 12:58:03 -04:00
mike cullerton 9c8ae36ba3 UVA IRB passes back `Q_COMPLETE` and `HSRNUMBER` 2021-11-03 12:40:43 -04:00
mike cullerton 7f163285c3 Test w/Alex's ID 2021-11-03 11:56:07 -04:00
mike cullerton 8be9a1968b configure logging 2021-11-03 11:21:39 -04:00
mike cullerton 50d4159168 info log didn't show up. Maybe debug? 2021-11-03 10:36:27 -04:00
mike cullerton 8b6c8f9518 Logging to troubleshoot API call 2021-11-03 10:14:11 -04:00
mike cullerton 1f9c80d70d Added ordered arguments to the get_localtime script
Modified the workflow so the first task is a form with booleans that allows me to test all the permutations.
Modified tests to cover all the permutations for calling the script
2021-11-03 08:36:24 -04:00
mike cullerton e22a9e7173 Cleanup, and added comments about the 2 sets of URLs 2021-11-02 15:32:43 -04:00
mike cullerton dd951b279b Migrated the remaining URLs 2021-11-02 14:06:34 -04:00
mike cullerton b80a2d181c Changed PB_BASE_URL and PB_USER_STUDIES_URL for testing move to production 2021-11-02 12:01:06 -04:00
mike cullerton 47ead5ab45 Fixed validation bug.
There is no element documentation available in the task, so we mock up the email content.
2021-11-02 10:00:41 -04:00
Dan Funk 8a6c1e1b51
Merge pull request #416 from sartography/bug/fix-hide-518
Fixes validation for #518
2021-11-01 16:22:24 -04:00
Dan Funk a4f8e39546
Merge pull request #415 from sartography/upload-file-previous-name-514
Upload file previous name #514
2021-11-01 15:59:57 -04:00
Dan Funk c4f3a3ad0d
Merge pull request #413 from sartography/logging-script-513
Logging script #513
2021-11-01 15:41:38 -04:00
Dan Funk fb25404a90
Merge pull request #412 from sartography/remove-content-html-512
Remove content_html #512
2021-11-01 15:38:46 -04:00
Dan Funk 8c5417d7f3
Merge pull request #409 from sartography/dependabot/pip/deploy/babel-2.9.1
Bump babel from 2.9.0 to 2.9.1 in /deploy
2021-11-01 15:36:52 -04:00
Dan Funk 28d4fa4675
Merge pull request #411 from sartography/convert-timestamp-499
get_localtime cleanup #499
2021-11-01 15:36:39 -04:00
alicia pritchett 1866e6bba9 Fixes validation for #518
Well really what this does is give every field a default value of None. what that does is allow you to use value expressions even without a given default value. this part in the backend fixes validation.
2021-11-01 14:35:30 -04:00