-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Hide advanced recipe options under expandable content #6021
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
Conversation
| </> | ||
| )} | ||
| {sessionId && ( | ||
| {sessionId && messages.length > 0 && ( |
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.
oh - what would this do before?
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.
It would show an empty recipe if you clicked it with no recipes.
I ended up pushing this change separately for the upcoming release but it wasn't a live bug in a current release, was on main from a previous refactor to move the recipe icon in the bottom bar.
|
This pull request has been automatically marked as stale because it has not had recent activity for 23 days. What happens next?
Thank you for your contribution! 🚀 |
Whitewolf111
left a comment
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.
Hi
* 'main' of github.com:block/goose: Claude 3.7 is out. we had some harcoded stuff (#6197) Release 1.19.0 chore: upgrade to node v24 as engine (#6361) chore(deps): bump rsa from 0.9.9 to 0.9.10 (#6358) Bump rust toolchain to 1.92 (current stable) (#6356) Hide advanced recipe options under expandable content (#6021) fix: use .config/agents (plural) for skills directory (#6357) fix: prevent KaTeX from treating underscores as subscripts in plain text (#6242) fix: make goose review PRs more like goose contributors do (#6240) fix : preserve provider engine type when editing custom providers (#6106) feat(providers): add retry for model fetching (#6347) allow goose issue solver to react to activation comments (#6239)

Summary
Visually simplify the recipe form by hiding the advanced options under expandable content
Expands automatically if there are advanced values when editing a recipe.
Also fixed a minor bug where the recipe icon in bottom bar was showing in an empty chat with no messages and improved the text a bit to not have ellipsis everywhere.
Added a link to the recipe docs also.