-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
fix: gui - WIP #3750
fix: gui - WIP #3750
Conversation
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Fixes and Improvements in GUI:
Fixed issue where agents were not listed on the home page.
Fixed issue preventing agent creation.
Fixed issue preventing agent edits from working.
Fixed start/stop agent API functionality.
Refactored the start/stop agent mechanism:
Previously, it checked the enabled flag in the database, which didn't really make sense to me 🤔. In this pr it checks the agent's runtime existence to determine whether to start or stop it.
In this PR, I removed some redundant components I noticed to simplify the UI and ensure everything functions correctly first.
UI/UX improvements can be considered in a future iteration.
TODO:
All UI functionalities work in this PR except chat.
Chat functionality will be addressed in a separate PR to keep this one manageable and easy to track.