Skip to content

fix: remove /preview/ prefix from frontend API calls#3924

Open
jangjohn wants to merge 4 commits intoAgenta-AI:mainfrom
jangjohn:fix/remove-preview-prefix
Open

fix: remove /preview/ prefix from frontend API calls#3924
jangjohn wants to merge 4 commits intoAgenta-AI:mainfrom
jangjohn:fix/remove-preview-prefix

Conversation

@jangjohn
Copy link
Copy Markdown

@jangjohn jangjohn commented Mar 5, 2026

fix: remove /preview/ prefix from frontend API calls

Closes #3831

Removes the /preview/ prefix from all frontend API calls across the web codebase, replacing them with canonical routes.

Changes

  • 61 files modified across web/oss/src/, web/packages/, and web/tests/
  • Only .ts and .tsx files changed
  • /preview/environments/ intentionally left unchanged (no canonical equivalent)

Notes


Open with Devin

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 5, 2026

@jangjohn is attempting to deploy a commit to the agenta projects Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 5, 2026
@dosubot dosubot bot added the Frontend label Mar 5, 2026
@jangjohn
Copy link
Copy Markdown
Author

jangjohn commented Mar 5, 2026

Hi @mmabrouk, this is a clean re-do of #3873. Re-cloned with proper line ending config so the diff only contains the intended .ts/.tsx route changes (61 files, no CSS or line ending noise). Ready for review!

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agenta-documentation Ready Ready Preview, Comment Mar 11, 2026 11:06am

Request Review

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional findings.

Open in Devin Review

devin-ai-integration[bot]

This comment was marked as resolved.

Copy link
Copy Markdown
Member

@mmabrouk mmabrouk left a comment

Choose a reason for hiding this comment

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

Thank you @jangjohn

Can you please fix linting for the CI to run correctly?

@jangjohn
Copy link
Copy Markdown
Author

jangjohn commented Mar 6, 2026

@mmabrouk I've also updated stale comments that still referenced /preview/ paths. Could you approve the workflows so CI can run? Thanks!

devin-ai-integration[bot]

This comment was marked as resolved.

@jangjohn
Copy link
Copy Markdown
Author

@mmabrouk Fixed the Prettier formatting issue and resolved the Devin review findings. Could you approve the workflows so CI can run? Thanks!

@jp-agenta jp-agenta self-assigned this Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Frontend: migrate endpoints to remove preview/ prefix

3 participants