Skip to content

Add install warning#3425

Closed
DOsinga wants to merge 72 commits intomainfrom
add-install-warning
Closed

Add install warning#3425
DOsinga wants to merge 72 commits intomainfrom
add-install-warning

Conversation

@DOsinga
Copy link
Copy Markdown
Collaborator

@DOsinga DOsinga commented Jul 15, 2025

Longer term this should be handled by the server, but that's true for the entire recipe processing, so leaving it here for now

@DOsinga DOsinga requested a review from zanesq July 15, 2025 10:08
wendytang and others added 24 commits July 17, 2025 18:48
rust failures are not related
Co-authored-by: Douwe Osinga <[email protected]>
…e/recipe-warning-modal

* 'add-install-warning' of github.com:block/goose:
  Link it and make it quit
  Move it to ipc
  Show the warning in the right place
  Undo
  WIP
  fix: wrap continueInitialization in useCallback and move after dependencies
  Goose Work

# Conflicts:
#	ui/desktop/openapi.json
#	ui/desktop/src/components/ChatView.tsx
#	ui/desktop/src/components/ui/RecipeWarningModal.tsx
#	ui/desktop/src/main.ts
#	ui/desktop/src/preload.ts
#	ui/desktop/src/utils/recipeHash.ts
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 20, 2025

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-07-22 15:34 UTC

@zanesq
Copy link
Copy Markdown
Contributor

zanesq commented Jul 20, 2025

@DOsinga my bad I messed up this branch I think we can close it in favor of #3537

@michaelneale
Copy link
Copy Markdown
Collaborator

woah don't know what is going on with github here - 450 files (!)

@michaelneale
Copy link
Copy Markdown
Collaborator

yeah sometimes I see this with forks - usualy to fix it you can target it to another branch, and then retarted it to main and it fixes up the history in github (but in this case- yeah that other PR is fine I think so can close this)

@DOsinga DOsinga closed this Jul 22, 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.