-
Notifications
You must be signed in to change notification settings - Fork 182
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(csharp): Add custom pager #6242
Conversation
…harp/custom-pager
…harp/custom-pager
…pi/fern into niels/csharp/custom-pager
…pi/fern into niels/csharp/custom-pager
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Main thing to look into is removing the type
for the offset and cursor pagination. All other comments are mostly nits.
[Obsolete("Use SendRequestAsync instead.")] | ||
public Task<ApiResponse> MakeRequestAsync( | ||
BaseApiRequest request, | ||
CancellationToken cancellationToken = default | ||
) | ||
{ | ||
return SendRequestAsync(request, cancellationToken); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The RawClient
is internal, so we can also just remove this entirely. We can also do that in a follow-up if you prefer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm overly cautious here for .fernignore'd files
generators/csharp/sdk/src/endpoint/http/HttpEndpointGenerator.ts
Outdated
Show resolved
Hide resolved
generators/csharp/sdk/src/endpoint/http/HttpEndpointGenerator.ts
Outdated
Show resolved
Hide resolved
… mock tests for custom pagination endpoint
Description
To do in follow-up PR, reconcile BiPager and Pager, and custom pager endpoints with normal pager endpoints (breaking change).
Changes Made
Testing