-
Notifications
You must be signed in to change notification settings - Fork 669
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
ui: introduce form injection for provider modals #1493
Conversation
2bf65ba
to
9adfd88
Compare
looks pretty reasonable - large one - what is best way to review? |
.bundle |
macOS ARM64 Desktop App (Apple Silicon)📱 Download macOS Desktop App (arm64, signed) Instructions: This link is provided by nightly.link and will work even if you're not logged into GitHub. |
* @param {string} [props.className] - Additional CSS classes for the main container | ||
* @returns {JSX.Element} | ||
*/ | ||
const CustomRadio = ({ |
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.
We should consider if we should use this, and if we can unify with what I had here for the top level of the settings page model radio buttons
goose/ui/desktop/src/components/settings_v2/SettingsView.tsx
Lines 90 to 96 in 2f00765
<input | |
type="radio" | |
name="model" | |
checked={model.selected} | |
onChange={() => handleModelSelect(model.id)} | |
className="h-4 w-4 text-white accent-[#393838] bg-[#393838] border-[#393838] checked:bg-[#393838] focus:ring-0 focus:ring-offset-0" | |
/> |
getActions
in provider registryno state related stuff -- tried but there's something not quite right with the config for providers (can't get keychain info), so next pr