Skip to content

Update button href to point to new endpoint#2849

Open
HarshMN2345 wants to merge 2 commits intomainfrom
HarshMN2345-patch-2
Open

Update button href to point to new endpoint#2849
HarshMN2345 wants to merge 2 commits intomainfrom
HarshMN2345-patch-2

Conversation

@HarshMN2345
Copy link
Copy Markdown
Member

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 2, 2026

Greptile Summary

This PR updates the "View full benchmark" button's href in ai-table.svelte from VARS.ARENA_ENDPOINT (the PUBLIC_ARENA_ENDPOINT environment variable) to the hardcoded URL https://arena.appwrite.io.

  • VARS.ARENA_ENDPOINT is the API backend endpoint used by src/routes/api/arena/+server.ts to fetch benchmark data, making it semantically incorrect for the user-facing button href.
  • The hardcoded https://arena.appwrite.io correctly targets the arena website, cleanly separating the concern of "where to fetch API data" from "where to send users."
  • Note: the PR description is entirely empty, providing no context for the change.

Confidence Score: 5/5

Safe to merge — simple one-line fix replacing an incorrectly used API endpoint variable with the correct user-facing URL.

The change is minimal and correct: VARS.ARENA_ENDPOINT is semantically an API backend URL (used as such in the arena server route with a .network fallback), not a website URL. Hardcoding https://arena.appwrite.io for the button is the right approach. No logic, data, or security concerns introduced.

No files require special attention.

Important Files Changed

Filename Overview
src/routes/(marketing)/(components)/ai-table.svelte Single-line change replacing VARS.ARENA_ENDPOINT (the API backend env var) with the hardcoded arena website URL https://arena.appwrite.io for the "View full benchmark" button href.

Reviews (2): Last reviewed commit: "Merge branch 'main' into HarshMN2345-pat..." | Re-trigger Greptile

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