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

Insert data to Mongo #37

Merged
merged 10 commits into from
Jul 6, 2016
Merged

Insert data to Mongo #37

merged 10 commits into from
Jul 6, 2016

Conversation

rachelhongca
Copy link
Member

created new struct for transcription data (same as Anosha's in pr #36)
wrote function to connect to database, write data

function to write data to data base
@rachelhongca
Copy link
Member Author

there's an issue with the struct, and we can't commit the godep changes...
@anoshaminai @sandlerben

rachelhongca added 2 commits April 9, 2016 13:00
@sandlerben
Copy link
Member

Once this is tested and working I will review the code

@yoninachmany
Copy link
Contributor

@rachelhongca any progress on understanding why checks have failed? The details of Travis say it's in make test but don't give a lot more than that

@sandlerben
Copy link
Member

I think the tests failing are because of withmock. I think we will switch how we do mock testing asap so don't let the failures dissuade us from merging this in.

@sandlerben sandlerben merged this pull request into master Jul 6, 2016
@sandlerben sandlerben deleted the insert-mongo branch July 6, 2016 01:19
sandlerben added a commit that referenced this pull request Jul 6, 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.

3 participants