-
Notifications
You must be signed in to change notification settings - Fork 164
feat: Save Anyways -> Save #8769
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?
Conversation
✅ Code Review CompleteSummaryThis PR successfully refactors "Save Anyway" to "Save" with improved visibility and adds the HighlightsArchitecture & Code Quality
Test Coverage
UX Improvements
Files Reviewed
LGTM! Ready to merge once CI passes. 🚀 🤖 Generated with Claude Code |
|
@lovincyrus, can you please do the first code review? Once you think it's good-to-go, please tag me and I'll do a final review. |
https://www.loom.com/share/cb6134ba1b484d4ab4b483edbe2b15da
Follow up to: #8636 (comment)

as Save Anyways caused mulitple keys to be created in .env.
Save is a step forward for "Edit connector" feature as it doesnt require a test connect to fail for the button to appear.
Checklist: