Skip to content

Feat: polishing#2

Merged
Philippoes merged 3 commits intomainfrom
feat/polishing
Jul 9, 2025
Merged

Feat: polishing#2
Philippoes merged 3 commits intomainfrom
feat/polishing

Conversation

@Philippoes
Copy link
Contributor

No description provided.

@Philippoes Philippoes requested review from Copilot and petar-omni and removed request for Copilot July 9, 2025 07:47
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR polishes the API client by renaming generated endpoint functions to shorter names, updating the default base URL, and enhancing the Orval transformer to strip redundant prefixes from operation names.

  • Rename all endpoint exports and corresponding result types to remove repetitive prefixes (e.g., yieldsGetYieldsgetYields).
  • Update customFetch default baseURL from api.stakek.it to api.yield.xyz.
  • Enhance orval.config.ts transformer to clean up and normalize operation names.

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/api/index.ts Renamed endpoint functions and result type aliases to simpler names.
src/api-client.ts Updated default baseURL in customFetch to new domain.
orval.config.ts Improved transformer logic to strip controller and common prefixes and ensure lowercase names.
example/index.ts Updated example call to use new function names.
README.md Updated code samples to reflect renamed endpoints and new base URL.
Comments suppressed due to low confidence (1)

README.md:76

  • This code block mixes TypeScript type annotations inside an object literal, which is invalid syntax. Consider replacing these annotations with concrete example values or separate the type signature from usage.
  network?: string;

@petar-omni
Copy link
Contributor

@Philippoes lets add CI action check as required before merging

@Philippoes Philippoes merged commit 703a315 into main Jul 9, 2025
3 checks passed
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