Skip to content

Conversation

@blackgirlbytes
Copy link
Contributor

BEFORE

Choosing Open Directory or New Session in the More Menu was not working because handleAction is not defined.

Screen.Recording.2024-12-22.at.9.26.17.PM.mov

AFTER

Remove handleAction from the onClick because that wrapper is unnecessary, so now it works.

Screen.Recording.2024-12-22.at.9.28.49.PM.mov

@blackgirlbytes blackgirlbytes self-assigned this Dec 23, 2024
@github-actions
Copy link
Contributor

Desktop App for this PR

The following build is available for testing:

The app is signed and notarized for macOS. After downloading, unzip the file and drag the Goose.app to your Applications folder.

This link is provided by nightly.link and will work even if you're not logged into GitHub.

@michaelneale michaelneale merged commit f632e61 into v1.0 Dec 23, 2024
6 checks passed
@michaelneale michaelneale deleted the fix/more-menu-directory-chooser branch December 23, 2024 22:07
michaelneale added a commit that referenced this pull request Dec 30, 2024
* v1.0:
  fix: Broken open directory and new session buttons in more menu (#520)
michaelneale added a commit that referenced this pull request Jan 6, 2025
* v1.0: (43 commits)
  feat: openrouter provider (#538)
  [ui] chore: tidy up gui providers (#537)
  [ui]: Polish and system theme fix (#533)
  [ui]: General ui polish to more closely match designs (#532)
  Latency issue fix with prepare_inference (#535)
  chore: use cross to build binaries (#507)
  feat: a system for non developers to augment developer system (#524)
  fix: Broken open directory and new session buttons in more menu (#520)
  refactor: move get_usage to provider trait (#506)
  fix: Make stop button more obvious (#516)
  fix: Enhance Dark mode menu dots visibility (#517)
  working on automating release of .zip and binaries and having them on each PR as well (#509)
  conditionally load memory system in goose server (#508)
  Adds 'instructions' field to InitializeResult (#511)
  feat: MCP client sdk (#505)
  Update cli-release.yml
  feat: added groq provider (#494)
  fix: use rust tls (#500)
  fix: Ldelalande/fix scroll (#504)
  feat: MCP server sdk (simple version first) (#499)
  ...
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