-
Notifications
You must be signed in to change notification settings - Fork 31
Add custom prompt template functionality #5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Appreciate the good work. Will review this soon! |
There was a problem hiding this 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 |
Recap/UseCases/Settings/ViewModels/General/GeneralSettingsViewModel.swift
Outdated
Show resolved
Hide resolved
Recap/UseCases/Settings/Components/TabViews/GeneralSettingsView.swift
Outdated
Show resolved
Hide resolved
Recap/UseCases/Settings/Components/TabViews/GeneralSettingsView.swift
Outdated
Show resolved
Hide resolved
Recap/UseCases/Settings/ViewModels/General/GeneralSettingsViewModel.swift
Outdated
Show resolved
Hide resolved
|
Please review the comments so we can move forward with merging |
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>
Uh oh!
There was an error while loading. Please reload this page.