Fix-realtime-settings #1344
Open
Fix-realtime-settings #1344
Conversation
This file contains 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
…logic - Add validation to skip realtime recording if no Deepgram API key is provided - Remove unused import in server core module - Fix indentation in server recording logic
- Update core and realtime modules to handle Deepgram API key and URL configuration - Allow realtime transcription with custom WebSocket URL when API key is not provided - Remove redundant API key validation in realtime streaming
|
@EzraEllette is attempting to deploy a commit to the louis030195's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
i think this is good but missing some UI changes on app settings, it's not intuitive atm |
|
@louis030195 What is not intuitive? The process is select screenpipe cloud as the realtime transcription engine and restart screenpipe. Maybe disable the selection unless either screenpipe-cloud is subscribed or a custom deepgram key is provided? |
|
so last time there was a user running real time transcription and he did not set api key or using screenpipe cloud without having a subscription or something like this and not until we looked into the logs we saw something was wrong it should show in the UI |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
description
related issue: #1335
how to test
Validating the custom url and key seems out of scope. I don't have screenpipe cloud so I can't test the authorization
/claim #1335