Add dependency injection registration for CloudConvertClient

Open Beginner friendly
#258 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
78/100
Issue type
Feature
Clarity
Clearly specified
Activity status
Active
Tech stack
azure, csharp
Domain
api, backend

Research direction

Open ConnectorServiceCollectionExtensions and compare the existing connector registration methods; then locate CloudconvertClient and ConnectorNames.CloudConvert. Add the matching registration tests and verify the client receives the configured runtime URL and managed-identity credential consistently with the other connectors.

Written by the indexing model from the issue text.

Description

Description

SDK 0.14 adds the generated CloudconvertClient and ConnectorNames.CloudConvert, but ConnectorServiceCollectionExtensions does not provide an AddCloudconvertClient method.

The samples use SDK DI registration extensions so clients receive their configured connection runtime URL and managed-identity credential. CloudConvert cannot follow that supported pattern even though its typed client is present.

Expected

Add AddCloudconvertClient(IServiceCollection, IConfiguration) using ConnectorNames.CloudConvert, consistent with the other connector registrations, and add registration tests.

A separate validation connection is still needed for deployed E2E coverage, but the missing DI extension is an SDK package gap.

Context

Found while expanding Azure/Connectors-NET-Samples#72 for SDK 0.14.

Authored with Dobby.

Dominant language
C#
Stars
3
Forks
5
Avg merge
2d 56m
Merged PRs (30d)
8

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from Azure/Connectors-NET-SDK

All issues in Azure/Connectors-NET-SDK

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.