Skip to content

fix(docs/tui): Reference to video assets#3637

Merged
abelanger5 merged 1 commit intohatchet-dev:mainfrom
gregfurman:fix/docs/tui-video-ref
Apr 23, 2026
Merged

fix(docs/tui): Reference to video assets#3637
abelanger5 merged 1 commit intohatchet-dev:mainfrom
gregfurman:fix/docs/tui-video-ref

Conversation

@gregfurman
Copy link
Copy Markdown
Collaborator

Description

Video assets being referenced in the TUI docs fail to load. This is due to redirect rules re-routing /cli paths to /reference/cli, causing the the frontend to look for video assets in public/reference/cli instead of public/cli.

See https://docs.hatchet.run/reference/cli/tui example:

image

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Documentation change (pure documentation change)

What's Changed

  • Moves video assets from public/cli/ to public/reference/cli -- mirroring the structure of repo and cirucumventing the redirects from /cli => /reference/cli i.e

    { source: '/cli/:path*', destination: '/reference/cli/:path*', permanent: false, basePath: false },

  • This should now return:

image

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 17, 2026

@gregfurman is attempting to deploy a commit to the Hatchet Team on Vercel.

A member of the Team first needs to authorize it.

@gregfurman gregfurman requested a review from grutt April 17, 2026 12:45
@gregfurman gregfurman marked this pull request as ready for review April 17, 2026 12:45
@gregfurman gregfurman requested a review from mrkaye97 April 17, 2026 16:24
@gregfurman gregfurman added the review: merge when ready Signals to the reviewer that a PR can be merged if accepted label Apr 19, 2026
@gregfurman gregfurman self-assigned this Apr 19, 2026
@abelanger5 abelanger5 merged commit 61aa161 into hatchet-dev:main Apr 23, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review: merge when ready Signals to the reviewer that a PR can be merged if accepted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants