Conversation
send only audio to assembly
Increase Hearing Scraper Timeout
Improve ffmpeg logic and file handling.
…e was ready, we were processing the in-progress livestream, which our transcription flow doesn't handle properly. Now, we should only process hearings after the video is finalized and we can process a complete transcription
Only Transcribe Full Videos
Make ffmpeg progress reporting less noisy.
… sub-optimal because we can't confirm that we've verified the email before sending, but sendgrid bounce management should cap the pain that can cause, users can't modify these message so the spam potential is severely limited, and the ongoing firebase auth credentials issue makes this check cause more pain than it resolves.
Use Profile Email Instead of Firebase Auth
…the template files from the deployed function, set a default text content for emails to avoid SendGrid errors
Fix Email Notification Bugs
Update Firestore Rules for Transcriptions
…function - we can manually trigger the scheduled function if need be, so this is useless. Also fixes a typo in the digest rendering that could prevent the template from rendering with data
…s the current notification system design and would actually be dangerous to run now
…n - we can always trigger the scheduled function manually if need be
Remove Old Notifications Functions
… now that they're hosted - these should ideally switch domain based on environment, but one problem at a time.
… to be applied in the final email without it, so I figure this won't hurt.
Switch email image links to the mapletestimony.org domain
…n based on user feedback, adding hyperlinks from bills to the respective bill page on MAPLE
…tments to header + bills, more adjustments to users, adding hyperlinks to user profiles + bill pages from the user section
Email Style Tweaks
Add sentence retreival and storage to transcription webhook.
… re-size columns for User Bills, center header text. Also fixes the test email script to accomodate the new pluralize helper
More Email Style Fixes
Mephistic
requested review from
alexjball,
kiminkim724,
mertbagt,
mvictor55,
nesanders,
sashamaryl and
timblais
as code owners
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary