-
Notifications
You must be signed in to change notification settings - Fork 1
Feat/UI #19
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
base: main
Are you sure you want to change the base?
Feat/UI #19
Conversation
…tabase function specified on board
…xclude Database.db and Migrations/
…tabase function specified on board
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 pull request introduces a new UI to the project using Spectre.Console while removing obsolete database components.
- Updated the project file to set the output type to Exe and add Spectre.Console and Spectre.Console.Cli package references.
- Refactored the main application into a UI-based structure with layout building and rendering in Program.cs and introduced a new Frontend/UI.cs for chat functionality.
- Removed the database models and SQLite persistence code.
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| sutor-aes/sutor-aes.csproj | Added output type specification and new UI library dependencies. |
| sutor-aes/Programcs.dis | Added encryption functions in a new file (note unconventional extension). |
| sutor-aes/Program.cs | Refactored to initialize and render the Spectre UI for the chat. |
| sutor-aes/Frontend/UI.cs | Added a comprehensive chat UI with input handling and live rendering. |
| Models/DBmodels.cs | Removed obsolete database models. |
| Database/SQLite.cs | Removed obsolete SQLite DB context implementation. |
Comments suppressed due to low confidence (3)
sutor-aes/Programcs.dis:1
- The file extension '.dis' is unconventional for C# source files; consider renaming it to '.cs' for clarity and consistency within the project.
using System.IO;
Models/DBmodels.cs:1
- Confirm that the removal of database models is intentional to avoid affecting features that may rely on these entities.
using System.ComponentModel.DataAnnotations;
Database/SQLite.cs:1
- Ensure that deleting the SQLite context is a planned change to prevent breaking any functionality that depends on the database integration.
using Microsoft.EntityFrameworkCore;
|
I only updated Frontend/UI.cs |
|
@IHtDzenda pls |
No description provided.