Update CrudExamplesTests.cs #8450

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

Jump to bottom

Open

RobinSoenen wants to merge 1 commit into elastic:8.17

from RobinSoenen:patch-1

Open

Update CrudExamplesTests.cs #8450

RobinSoenen wants to merge 1 commit into elastic:8.17 from RobinSoenen:patch-1

Conversation

RobinSoenen

Copy link

@RobinSoenen RobinSoenen commented Jan 29, 2025

Small change in the documentation:

var response = await client.IndexAsync(tweet, "my-tweet-index"); // <2>

To
var response = await client.IndexAsync(tweet, (IndexName)"my-tweet-index"); // <2>

in order to avoid an ambiguous call:

error CS0121: The call is ambiguous between the following methods or properties: 'ElasticsearchClient.IndexAsync(TDocument, IndexName, CancellationToken)' and 'ElasticsearchClient.IndexAsync(TDocument, Id?, CancellationToken)'

@RobinSoenen


 Update CrudExamplesTests.cs

8096f66

Small change in the documentation:
 
var response = await client.IndexAsync(tweet, "my-tweet-index"); // <2>
To 
var response = await client.IndexAsync(tweet, (IndexName)"my-tweet-index"); // <2>
in order to avoid an ambiguous call:
error CS0121: The call is ambiguous between the following methods or properties: 'ElasticsearchClient.IndexAsync(TDocument, IndexName, CancellationToken)' and 'ElasticsearchClient.IndexAsync(TDocument, Id?, CancellationToken)'

@cla-checker-service CLA Checker Service

Copy link