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

Add support for audio file PDFs and alternate transcript receipt format #326

Merged
merged 1 commit into from
Jun 29, 2023

Conversation

ttys0dev
Copy link
Contributor

Fixes #244.

@CLAassistant
Copy link

CLAassistant commented Apr 29, 2023

CLA assistant check
All committers have signed the CLA.

@ttys0dev ttys0dev force-pushed the audio branch 2 times, most recently from 7e3f138 to e4bc000 Compare April 29, 2023 22:42
@ttys0dev ttys0dev changed the title Add support for audio file PDFs Add support for audio file PDFs and alternate transcript receipt format Apr 29, 2023
src/pacer.js Outdated Show resolved Hide resolved
@ttys0dev ttys0dev force-pushed the audio branch 3 times, most recently from 77223e0 to 8785d90 Compare April 30, 2023 23:50
@mlissner
Copy link
Member

mlissner commented May 1, 2023

Wow, busy weekend. It looks like this one is ready too?

@ttys0dev
Copy link
Contributor Author

ttys0dev commented May 2, 2023

It looks like this one is ready too?

Yep.

@mlissner
Copy link
Member

mlissner commented May 2, 2023

Great. I'm putting it on the top of @ERosendo's backlog to review, but he's on a high priority project right now so he might be a moment until he can come up for air. But we do try to prioritize PRs, so hopefully he can switch to it once his current thing is In Review.

@mlissner
Copy link
Member

mlissner commented Jun 9, 2023

Darn, we should have tried to get this into yesterday's release. @ERosendo can you try to get this some love when you finish your current task, please?

@ERosendo
Copy link
Contributor

ERosendo commented Jun 9, 2023

Sure!

Copy link
Contributor

@ERosendo ERosendo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. I tested your code by uploading a few audio files from 4:17-cr-03038 and It worked great.

Just one comment.

Thanks!

src/pacer.js Show resolved Hide resolved
@mlissner mlissner merged commit 05a7821 into freelawproject:main Jun 29, 2023
@ttys0dev ttys0dev deleted the audio branch June 29, 2023 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

RECAP Fails on Audio Files
4 participants