Skip to content

Conversation

@lily-de
Copy link
Contributor

@lily-de lily-de commented Apr 2, 2025

When testing with radio buttons found that they weren't selecting properly

try {
const result = await getCurrentModelAndProvider({ readFromConfig: read });
if (isMounted) {
console.log(modelList);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: remove log statements if no longer needed

try {
const result = await getCurrentModelAndProvider({ readFromConfig: read });
if (isMounted) {
console.log(modelList);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
console.log(modelList);

@lily-de lily-de merged commit 4e5989d into main Apr 2, 2025
6 checks passed
@lily-de lily-de deleted the ldelalande/fix-radio-button-selection branch April 2, 2025 13:02
laanak08 added a commit that referenced this pull request Apr 3, 2025
* main:
  fix: handle the case sensitive in mac screenshot name (#2021)
  ui: better env var ux (#2006)
  feat: add ANTHROPIC_HOST configuration for Anthropic in cli and UI (#1776)
  Fix function params  (#2012)
  revert: "feat: allowlist optionally for goosed (#1848)"  (#2010)
  # feat(providers): Add support for generic GCP Vertex AI Claude and Gemini models (#1909)
  fix: fix empty result for view tool response (#2011)
  Blog: Top MCP servers I use  (#1951)
  docs: MCP for Nondevs Blog (#1910)
  styles: update markdown styles (#2005)
  fix(ollama): respect ollama URL and port (#2004)
  ui: refresh selected model (#2002)
  ui: fix radio button selection (#2001)
  use provider display names (#2000)
  ui: settings v2 click anywhere to exit models bottom bar (#1997)
  feat: better ErrorBoundary UI (#1993)
  feat: check tool annotation before permission check (#1957)
  fix: see last msg fully in shared session view (#1994)
ahau-square pushed a commit that referenced this pull request May 2, 2025
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants