Skip to content

fix(inferx): update api endpoint to endpoints/v1#3252

Open
Prashanth-InferX wants to merge 1 commit into
anomalyco:devfrom
Prashanth-InferX:patch-2
Open

fix(inferx): update api endpoint to endpoints/v1#3252
Prashanth-InferX wants to merge 1 commit into
anomalyco:devfrom
Prashanth-InferX:patch-2

Conversation

@Prashanth-InferX

@Prashanth-InferX Prashanth-InferX commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

This PR updates the base API URL to point directly to our endpoint routing at https://model.inferx.net/endpoints/v1. This change aligns the registry calls with our actual backend endpoint structure.

You can verify the valid endpoint configuration base under our official developer documentation here: https://model.inferx.net/endpoints

@github-actions

Copy link
Copy Markdown
Contributor

Action items

  • [low] [possible mistake] providers/inferx/provider.toml:4 - Check: Provider API endpoint change requires a supporting citation. Why: The PR body is empty and the api value is changed from https://model.inferx.net/v1 to https://model.inferx.net/endpoints/v1, a material factual change that determines the base URL consumers use to call InferX. The new value is internally consistent with the existing doc = "https://model.inferx.net/endpoints" and the author is affiliated with InferX, but no first-party API reference is cited to confirm the exact /endpoints/v1 path. Action: Add a link to InferX's API documentation (the existing doc URL or an API reference) in the PR body that confirms the correct base endpoint, so the change can be reviewed against the provider's own specification.

@Prashanth-InferX

Copy link
Copy Markdown
Contributor Author

Thanks so much for merging the initial provider setup earlier—really appreciate it! This is just a quick, one-line configuration update to align the API path with our direct /endpoints/v1 routing so everything runs smoothly. Ready for review when you have a quick second!

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.

1 participant