fix: Address AI code reviewer feedback
- Fix branding references (VoiceLink → VoiceInk) in documentation - Correct keyboard shortcut documentation to Cmd+Shift+/ for proper Cmd+? - Remove force unwrap in webhook example code Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
This commit is contained in:
parent
d798a99994
commit
93a790833e
@ -7,7 +7,7 @@
|
||||
|
||||
## What Was Implemented
|
||||
|
||||
We successfully implemented **5 critical quality of life improvements** for VoiceLink Community:
|
||||
We successfully implemented **5 critical quality of life improvements** for VoiceInk:
|
||||
|
||||
### ✅ 1. Recording Duration Indicator
|
||||
- Real-time timer showing MM:SS format during recording
|
||||
@ -381,4 +381,4 @@ Before merging/deploying:
|
||||
|
||||
**Last Updated:** November 3, 2025
|
||||
**Implemented By:** AI Assistant via Factory
|
||||
**Maintained By:** VoiceLink Community
|
||||
**Maintained By:** VoiceInk Community
|
||||
|
||||
@ -239,7 +239,7 @@ struct KeyboardShortcutCheatSheet: View {
|
||||
Button("Keyboard Shortcuts") {
|
||||
NotificationCenter.default.post(name: .showShortcutCheatSheet, object: nil)
|
||||
}
|
||||
.keyboardShortcut("/", modifiers: .command)
|
||||
.keyboardShortcut("/", modifiers: [.command, .shift])
|
||||
}
|
||||
}
|
||||
```
|
||||
|
||||
@ -823,7 +823,11 @@ func sendToWebhook(_ transcription: Transcription) async throws {
|
||||
"duration": transcription.duration
|
||||
]
|
||||
|
||||
var request = URLRequest(url: URL(string: settings.url)!)
|
||||
guard let url = URL(string: settings.url) else {
|
||||
throw WebhookError.invalidURL
|
||||
}
|
||||
|
||||
var request = URLRequest(url: url)
|
||||
request.httpMethod = "POST"
|
||||
request.httpBody = try JSONSerialization.data(withJSONObject: payload)
|
||||
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user