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

createTranscript Enhancement #90

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hargup
Copy link
Collaborator

@hargup hargup commented Jan 6, 2015

Signed-off-by: Harsh Gupta [email protected]

Signed-off-by: Harsh Gupta <[email protected]>
@hargup hargup force-pushed the create_transcript_enhancement branch from ea6a068 to f10390b Compare January 6, 2015 13:02
@hargup
Copy link
Collaborator Author

hargup commented Jan 17, 2015

@yeban can you review this PR?

@yeban
Copy link
Contributor

yeban commented Feb 3, 2015

I'm sorry I couldn't get back earlier. Was caught up in other things. I had a quick look - seems good to me. But I will wait to merge till I have had a better look - should be later this week or the next. At this point I can say it would be safe to base further work (more advanced test suite) on this PR (after having rebased it on master).

Thanks.

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.

2 participants