-
-
Notifications
You must be signed in to change notification settings - Fork 582
Features/refine model settings #1168
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
Features/refine model settings #1168
Conversation
…atures/refine-model-settings
…atures/refine-model-settings
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨Explore these optional code suggestions:
|
PR Type
Enhancement
Description
Add intermediate message support for chat events
Refactor database configuration in ExcelHandler plugin
Remove dependency on SqlDriver plugin
Update file selection model structure
Diagram Walkthrough
File Walkthrough
8 files
Add intermediate message chat event
Remove filename property from FileSelectItem
Add SaveDataToDb property with documentation
Add intermediate message emission in debug mode
Refactor PDF to image conversion logic
Simplify image converter retrieval method
Remove instruction hook emission code
Add intermediate message handling with database save
2 files
Change class visibility to internal
Add ExcelHandlerSettings to global usings
6 files
Update database provider configuration access
Replace SqlDriver dependency with local settings
Replace SqlDriver dependency with local settings
Add DatabaseSettings class with provider and connection
Remove temporary connection string properties
Update ExcelHandler configuration structure
1 files
Update router instruction with default language
1 files
Remove SqlDriver project reference