Skip to content

Update CrudExamplesTests.cs#8450

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

Update CrudExamplesTests.cs#8450
RobinSoenen wants to merge 1 commit into
elastic:8.17from
RobinSoenen:patch-1

Conversation

@RobinSoenen
Copy link
Copy Markdown

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)'

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
Copy link
Copy Markdown

cla-checker-service Bot commented Jan 29, 2025

💚 CLA has been signed

@RobinSoenen
Copy link
Copy Markdown
Author

Contributor agreement signed in the meantime

@RobinSoenen RobinSoenen reopened this Jan 29, 2025
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