-
-
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 eventRemove filename property from FileSelectItemAdd SaveDataToDb property with documentationAdd intermediate message emission in debug modeRefactor PDF to image conversion logicSimplify image converter retrieval methodRemove instruction hook emission codeAdd intermediate message handling with database save2 files
Change class visibility to internalAdd ExcelHandlerSettings to global usings6 files
Update database provider configuration accessReplace SqlDriver dependency with local settingsReplace SqlDriver dependency with local settingsAdd DatabaseSettings class with provider and connectionRemove temporary connection string propertiesUpdate ExcelHandler configuration structure1 files
Update router instruction with default language1 files
Remove SqlDriver project reference