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

Surveys through redcap #581

Closed

Conversation

squirrelo
Copy link
Contributor

@squirrelo squirrelo commented Mar 17, 2016

This takes care of the actual survey logging and back and forth between redcap and AG. Needs some way of testing this communication, however.

Requires #579 Merged

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.9%) to 58.54% when pulling e2abeeb on squirrelo:surveys-through-redcap into 84d2c61 on biocore:redcap-integration.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 0286541 on squirrelo:surveys-through-redcap into * on biocore:redcap-integration*.

@squirrelo squirrelo changed the title WIP - Surveys through redcap Surveys through redcap May 16, 2016
@squirrelo
Copy link
Contributor Author

@EmbrietteH this is ready for review.

@EmbrietteH
Copy link
Contributor

ping @wasade

@wasade
Copy link
Member

wasade commented May 17, 2016

Did a static analysis and looks with except missing tests for create_record and as_transaction. I'm particularly concerned about positive/negative tests for as_transaction in the context of a tornado handler, and whether failure conditions muck with tornado. Would it be possible to add a test or two?

@wasade wasade closed this Sep 29, 2016
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.

None yet

4 participants