Skip to content

Conversation

@blackgirlbytes
Copy link
Contributor

Problem

The Docker tutorial documentation contained broken links to hardcoded_stuff.tsx which was removed in commit a89fa18 (PR #2744). These links were returning 404 errors when users tried to access them.

Solution

Updated the broken links to point to the current ProviderRegistry.tsx file, which contains the same provider and API key information that users need for Docker configuration.

Changes

  • Fixed broken GitHub links in documentation/docs/tutorials/goose-in-docker.md
  • Updated link text to be more descriptive
  • Verified new link works and points to correct provider registry

Before/After

Before:

  • Link to non-existent hardcoded_stuff.tsx (404 error)

After:

  • Link to current ProviderRegistry.tsx (working link with same information)

This ensures users can actually access the provider and API key information they need when setting up Goose in Docker.

The hardcoded_stuff.tsx file was removed in commit a89fa18 but the Docker
tutorial documentation still referenced it. Updated the links to point to the
current ProviderRegistry.tsx file which contains the same provider and API key
information that users need for Docker configuration.

Fixes broken links that were returning 404 errors.
@blackgirlbytes blackgirlbytes requested a review from a team as a code owner August 22, 2025 16:33
@github-actions
Copy link
Contributor

github-actions bot commented Aug 22, 2025

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-08-22 16:58 UTC

@blackgirlbytes blackgirlbytes merged commit 2d030fa into main Aug 22, 2025
12 checks passed
@blackgirlbytes blackgirlbytes deleted the fix/docker-docs-broken-links branch August 22, 2025 16:57
michaelneale added a commit that referenced this pull request Aug 24, 2025
* main:
  docs: Plan tutorial (#4309)
  Extensions Modal Improvements (#4293)
  docs: fixed cicd tutorial pipeline in docs (#4223)
  Read oltp config from config and env (#4292)
  release/1.6.0 (#4280)
  docs: fix broken links in Docker tutorial (#4285)
  Remove half-second wait, rework auto submit (#4282)
  Block send until extensions are ready (#4271)
  fix: improve OpenAI-compatible error handling and add test coverage (#4175)
  Move To-Do Tool to Session Scope from Agent Scope (#4157)
  fix: recipe params not being replaced all the time (#4207)
  chore: removing little-used session sharing feature (#4249)
  Stop auto scrolling when agent responds and let scroll area handle scrolling to bottom (#4257)
  restore cli projects from accidental removal during cleanup (#4266)
  Fix: deep link extension installation to show dialog for headers configuration (#4150)
michaelneale added a commit that referenced this pull request Aug 25, 2025
* main: (24 commits)
  feat: autovisualiser of structured data with mcp-ui (#4153)
  docs: Plan tutorial (#4309)
  Extensions Modal Improvements (#4293)
  docs: fixed cicd tutorial pipeline in docs (#4223)
  Read oltp config from config and env (#4292)
  release/1.6.0 (#4280)
  docs: fix broken links in Docker tutorial (#4285)
  Remove half-second wait, rework auto submit (#4282)
  Block send until extensions are ready (#4271)
  fix: improve OpenAI-compatible error handling and add test coverage (#4175)
  Move To-Do Tool to Session Scope from Agent Scope (#4157)
  fix: recipe params not being replaced all the time (#4207)
  chore: removing little-used session sharing feature (#4249)
  Stop auto scrolling when agent responds and let scroll area handle scrolling to bottom (#4257)
  restore cli projects from accidental removal during cleanup (#4266)
  Fix: deep link extension installation to show dialog for headers configuration (#4150)
  feat: Add message queue system with interruption handling (#4179)
  Start extensions concurrently  (#4234)
  Add X-Title and referer headers on exchange to tetrate (#4250)
  docs: update View/Edit Recipe menu item name (#4267)
  ...
shellz-n-stuff pushed a commit to shellz-n-stuff/goose that referenced this pull request Aug 27, 2025
dorien-koelemeijer pushed a commit to dorien-koelemeijer/goose that referenced this pull request Sep 2, 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