Skip to content

Conversation

@dennisimoo
Copy link
Contributor

@dennisimoo dennisimoo commented Aug 5, 2025

Screenshot 2025-08-04 at 11 11 31 PM

@rawandahmad698
Copy link
Collaborator

Appreciate the good work. Will review this soon!

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds custom prompt template functionality to the settings UI, allowing users to customize AI summarization prompts for meeting transcripts.

  • Adds custom prompt template management with save/reset functionality
  • Creates a new reusable CustomTextEditor component for multi-line text input
  • Integrates the custom prompt template editor into the general settings view

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
GeneralSettingsViewModelType.swift Adds protocol methods for custom prompt template management
GeneralSettingsViewModel.swift Implements custom prompt template persistence and business logic
SettingsView.swift Updates preview view model to support new custom prompt functionality
GeneralSettingsView.swift Integrates custom prompt template UI into the settings interface
CustomTextEditor.swift New reusable text editor component with styling and focus management

@rawandahmad698
Copy link
Collaborator

Please review the comments so we can move forward with merging

dennisimoo and others added 5 commits August 5, 2025 22:58
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Rawand Ahmed Shaswar <rawandahmad698@gmail.com>
@rawandahmad698 rawandahmad698 merged commit 21dd338 into RecapAI:main Aug 6, 2025
2 of 4 checks 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