Skip to content
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

Add documentation for using DataserviceContext IHttpClientFactory in OData Client #340

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

WanjohiSammy
Copy link
Contributor

@WanjohiSammy WanjohiSammy commented Nov 27, 2024

Description

This PR adds comprehensive documentation on how to use IHttpClientFactory with Microsoft.OData.Client.

View the Doc here

@WanjohiSammy
Copy link
Contributor Author

WanjohiSammy commented Nov 27, 2024

@microsoft-github-policy-service agree [company="Microsoft"]

Copy link
Contributor

Learn Build status updates of commit 779fe91:

⚠️ Validation status: warnings

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ⚠️Warning View Details

Odata-docs/client/using-dataservice-ihttpclientfactory.md

  • Line 1, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.
  • Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
  • Line 77, Column 3: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/odata/client/using-httpclient' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 1, Column 1: [Suggestion: ms-date-missing - See documentation] Missing required attribute: 'ms.date'.
  • Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
  • Line 77, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/odata/client/using-httpclient' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@WanjohiSammy
Copy link
Contributor Author

@microsoft-github-policy-service agree
@microsoft-github-policy-service agree company="Microsoft"

Copy link
Contributor

Learn Build status updates of commit bdcacc8:

⚠️ Validation status: warnings

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ⚠️Warning View Details

Odata-docs/client/using-dataservice-ihttpclientfactory.md

  • Line 1, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.
  • Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
  • Line 1, Column 1: [Suggestion: ms-date-missing - See documentation] Missing required attribute: 'ms.date'.
  • Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
  • Line 3, Column 38: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/odata/client/getting-started' will be broken in isolated environments. Replace with a relative link.
  • Line 7, Column 1: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/dotnet/core/extensions/httpclient-factory' will be broken in isolated environments. Replace with a relative link.
  • Line 18, Column 17: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/dotnet/core/extensions/dependency-injection' will be broken in isolated environments. Replace with a relative link.
  • Line 82, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/odata/client/using-httpclient' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@WanjohiSammy WanjohiSammy requested a review from habbes November 29, 2024 13:17
Copy link
Contributor

Learn Build status updates of commit dd13c64:

⚠️ Validation status: warnings

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ⚠️Warning View Details

Odata-docs/client/using-dataservice-ihttpclientfactory.md

  • Line 7, Column 10: [Warning: ms-date-invalid - See documentation] Invalid date format for 'ms.date': '29/11/2024'. If specified, must be a date in format M/D/YYYY, no more than 30 days from today. If you don't specify ms.date, the date will be updated automatically.
  • Line 14, Column 38: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/odata/client/getting-started' will be broken in isolated environments. Replace with a relative link.
  • Line 18, Column 1: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/dotnet/core/extensions/httpclient-factory' will be broken in isolated environments. Replace with a relative link.
  • Line 29, Column 17: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/dotnet/core/extensions/dependency-injection' will be broken in isolated environments. Replace with a relative link.
  • Line 93, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/odata/client/using-httpclient' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@WanjohiSammy WanjohiSammy force-pushed the swanjohi/use-dataservicecontext-ihttpclientfactory branch from dd13c64 to 3075b06 Compare November 29, 2024 16:43
Copy link
Contributor

Learn Build status updates of commit 3075b06:

✅ Validation status: passed

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 5499ce8:

⚠️ Validation status: warnings

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ⚠️Warning View Details

Odata-docs/client/using-dataservice-ihttpclientfactory.md

  • Line 5, Column 9: [Warning: author-not-found - See documentation] Invalid value for author: 'Wanjohi Sammy' is not a valid GitHub ID.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@WanjohiSammy WanjohiSammy requested a review from habbes December 6, 2024 08:29
Copy link
Contributor

Learn Build status updates of commit e3e5f70:

⚠️ Validation status: warnings

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ⚠️Warning View Details

Odata-docs/client/using-dataservice-ihttpclientfactory.md

  • Line 5, Column 9: [Warning: author-not-found - See documentation] Invalid value for author: 'Wanjohi Sammy' is not a valid GitHub ID.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 887f1ef:

⚠️ Validation status: warnings

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ⚠️Warning View Details

Odata-docs/client/using-dataservice-ihttpclientfactory.md

  • Line 5, Column 9: [Warning: author-not-found - See documentation] Invalid value for author: 'Wanjohi Sammy' is not a valid GitHub ID.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 0071bdd:

✅ Validation status: passed

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 6493ffe:

✅ Validation status: passed

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 2f7b2bc:

✅ Validation status: passed

File Status Preview URL Details
Odata-docs/client/using-dataservice-ihttpclientfactory.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

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.

3 participants