Skip to content

Conversation

@amed-xyz
Copy link
Collaborator

Mostly putting things into one central place (ExtensionInstallModal) to reduce fragmentation.

@amed-xyz amed-xyz self-assigned this Aug 25, 2025
@amed-xyz amed-xyz marked this pull request as ready for review August 25, 2025 20:00
@amed-xyz amed-xyz changed the title Refactor Extensions Modal Refactor Extensions Install Modal Aug 26, 2025
Copy link
Collaborator

@DOsinga DOsinga left a comment

Choose a reason for hiding this comment

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

Looks much better. I'd still hold back on writing this sort of tests since I dont know how they will guard for anything, but let's go with it

@amed-xyz amed-xyz merged commit a3097f5 into main Aug 26, 2025
11 checks passed
@amed-xyz amed-xyz deleted the amed/refactor-ExtensionInstallModal branch August 26, 2025 20:28
shellz-n-stuff pushed a commit to shellz-n-stuff/goose that referenced this pull request Aug 27, 2025
Signed-off-by: Alex Rosenzweig <arosenzweig@squareup.com>
zanesq added a commit that referenced this pull request Aug 27, 2025
…nscribing

* 'main' of github.com:block/goose:
  update tetrate blog date to today (#4368)
  tetrate signup: blog/launch post (#4313)
  Implement graceful recipe error handling with filename display (#4363)
  docs: airgapped operation by bypassing hermit for desktop app (#4063)
  remove Ollama card from welcome screen (#4348)
  feat: initial implementation of extension malware check (#4272)
  Add Tetrate Agent Router Service to Provider Registry (#4354)
  Goose Simple Compact UX (#4202)
  Refactor Extensions Install Modal (#4328)
  fix: url path trailing slash for custom-providers (#4345)
  docs: update available and onboarding providers list (#4356)
katzdave added a commit that referenced this pull request Aug 27, 2025
* 'main' of github.com:block/goose:
  Refactor Extensions Install Modal (#4328)
  fix: url path trailing slash for custom-providers (#4345)
  docs: update available and onboarding providers list (#4356)
  Fix refresh by waiting for document ready rather than a timeout (#4355)
  docs: add Tetrate Agent Router Service to available providers (#4339)
michaelneale added a commit that referenced this pull request Aug 28, 2025
* main: (38 commits)
  feat: linux computer control for android (termux) (#3890)
  feat: Added scroll state support for chat-session-list navigation (#4360)
  docs: typo fix (#4376)
  blog: goose janitor (#4131)
  Fix eleven labs audio transcription and added more logging (#4358)
  feat: re-introduce session sharing (#4370)
  remove duplicate blog post (#4369)
  fix focus ring under form submits (#4332)
  Trigger docs deployment
  update tetrate blog date to today (#4368)
  tetrate signup: blog/launch post (#4313)
  Implement graceful recipe error handling with filename display (#4363)
  docs: airgapped operation by bypassing hermit for desktop app (#4063)
  remove Ollama card from welcome screen (#4348)
  feat: initial implementation of extension malware check (#4272)
  Add Tetrate Agent Router Service to Provider Registry (#4354)
  Goose Simple Compact UX (#4202)
  Refactor Extensions Install Modal (#4328)
  fix: url path trailing slash for custom-providers (#4345)
  docs: update available and onboarding providers list (#4356)
  ...
dorien-koelemeijer pushed a commit to dorien-koelemeijer/goose that referenced this pull request Sep 2, 2025
Signed-off-by: Dorien Koelemeijer <dkoelemeijer@squareup.com>
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.

2 participants