Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

feat(desktop): add upload indicator for audio file uploads

Summary

Adds visual feedback when uploading audio files in the desktop app. Previously, clicking "Upload audio" would show no indication that anything was happening, leaving users confused about whether the upload was in progress.

Changes:

  • Added isUploading state to track upload progress
  • Added check for isBatchProcessing from listener store to track batch transcription
  • Shows spinning loader icon instead of ellipsis menu icon during processing
  • Displays tooltip with "Uploading audio..." or "Processing audio..." status
  • Disables the options menu button during processing to prevent duplicate uploads

Review & Testing Checklist for Human

  • Test the upload flow end-to-end: Select an audio file and verify the loading indicator appears immediately and persists until processing completes
  • Test with no STT provider configured: The underlying useRunBatch hook returns early with only a console.error if no STT connection exists - verify the indicator still disappears and consider if user-facing error feedback is needed
  • Test canceling the file dialog: Ensure the loading state doesn't get stuck if user cancels file selection
  • Verify batch processing indicator: When batch transcription is running, the indicator should show "Processing audio..."

Test Plan

  1. Open a session in the desktop app
  2. Click the ellipsis menu button on the floating action button
  3. Click "Upload audio" and select an audio file
  4. Verify: Loading spinner appears with "Uploading audio..." tooltip
  5. Verify: After upload completes, spinner changes to "Processing audio..." during batch transcription
  6. Verify: Button returns to normal state after processing completes

Notes

  • This PR adds the visual indicator but does not change the underlying upload/batch processing logic
  • If uploads are failing silently due to missing STT configuration, users will now at least see the indicator disappear (though no error message is shown)

Requested by: @ComputelessComputer

Link to Devin run: https://app.devin.ai/sessions/633f74198e174d3a98cbc10d15df39c7

- Add isUploading state to track when audio file is being uploaded
- Show loading spinner icon when uploading or batch processing
- Add tooltip showing upload/processing status
- Disable options menu button during processing to prevent duplicate uploads

Co-Authored-By: john@hyprnote.com <john@hyprnote.com>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@netlify
Copy link

netlify bot commented Jan 7, 2026

Deploy Preview for hyprnote canceled.

Name Link
🔨 Latest commit 43cc156
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/695e996590e0190008a583e2

@netlify
Copy link

netlify bot commented Jan 7, 2026

Deploy Preview for hyprnote-storybook canceled.

Name Link
🔨 Latest commit 43cc156
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/695e9965df863b0008cb3bc2

@netlify
Copy link

netlify bot commented Jan 7, 2026

Deploy Preview for howto-fix-macos-audio-selection canceled.

Name Link
🔨 Latest commit 43cc156
🔍 Latest deploy log https://app.netlify.com/projects/howto-fix-macos-audio-selection/deploys/695e99655837f40008c818ea

@ComputelessComputer ComputelessComputer deleted the devin/1767806741-fix-audio-upload-indicator branch January 9, 2026 02:38
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