fix: remove /preview/ prefix from frontend API calls#3924
fix: remove /preview/ prefix from frontend API calls#3924jangjohn wants to merge 2 commits intoAgenta-AI:mainfrom
Conversation
|
@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
bot
added
the
size:L
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
There was a problem hiding this comment.
(Refers to line 22)
📝 Info: Stale JSDoc comments still reference /preview/ paths
Several files retain comments referencing the old /preview/ paths that are now outdated. For example, web/oss/src/services/evaluationRuns/api/index.ts:292 still says the POST /preview/evaluations/runs/ endpoint, web/oss/src/components/TestcasesTableNew/hooks/api.ts:22 references /preview/testcases/query, and web/oss/src/state/entities/testset/revisionSchema.ts:5 references /preview/testsets/revisions/query. These are documentation inconsistencies, not functional bugs, but should ideally be updated for developer clarity.
Was this helpful? React with 👍 or 👎 to provide feedback.
|
@mmabrouk I've also updated stale comments that still referenced /preview/ paths. Could you approve the workflows so CI can run? Thanks! |
2 participants
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
web/oss/src/,web/packages/, andweb/tests/.tsand.tsxfiles changed/preview/environments/intentionally left unchanged (no canonical equivalent)Notes