Skip to content

Conversation

@marksamfd
Copy link
Owner

@marksamfd marksamfd commented Aug 20, 2025

Added

  • Error Monitoring:
    • Integrated Sentry in both main and renderer processes
    • Configured renderer to send default PII and custom tags

Fixed

  • App Stability:
    • Guarded against undefined currentProject during save and close operations
    • Updated window close handling to prevent errors when no project is open
  • File Handling:
    • Improved video file picker with null-safety checks
    • Fixed video filename parsing logic to correctly strip extensions
    • Enhanced media responder to return 404 Not Found for missing files
  • UX Improvements:
    • Changed file save dialog message from "File Is Saving""File Saved"

Technical Details

  • Removed unnecessary debug logging (console.log, drain, data, fileCreator outputs)
  • Simplified video filename handling logic in Slide.ts
  • Added safer stream handling in MediaResponderClass

### Added
- **Error Monitoring**:
  - Integrated Sentry in both `main` and `renderer` processes
  - Configured renderer to send default PII and custom tags
  - Hardcoded DSN initialization for main process

### Fixed
- **App Stability**:
  - Guarded against undefined `currentProject` during save and close operations
  - Updated window close handling to prevent errors when no project is open
- **File Handling**:
  - Improved video file picker with null-safety checks
  - Fixed video filename parsing logic to correctly strip extensions
  - Enhanced media responder to return `404 Not Found` for missing files
- **UX Improvements**:
  - Changed file save dialog message from `"File Is Saving"` → `"File Saved"`

### Technical Details
- Removed unnecessary debug logging (`console.log`, `drain`, `data`, `fileCreator` outputs)
- Simplified video filename handling logic in `Slide.ts`
- Added safer stream handling in `MediaResponderClass`
@marksamfd marksamfd merged commit bdd2ac7 into main Aug 20, 2025
1 check passed
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