Skip to content

Add multi-tenant guidance for .NET#1056

Merged
amerjusupovic merged 6 commits intomainfrom
ajusupovic/dotnet-multitenant-info
May 27, 2025
Merged

Add multi-tenant guidance for .NET#1056
amerjusupovic merged 6 commits intomainfrom
ajusupovic/dotnet-multitenant-info

Conversation

@amerjusupovic
Copy link
Copy Markdown
Contributor

@amerjusupovic amerjusupovic commented May 20, 2025

Inspired by this comment.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds guidance for configuring multi-tenant applications in .NET and corrects minor typos in the WebDemo example.

  • Fixed spelling of "overridden" in cache expiration comments for Azure App Configuration and feature flags.
  • Introduced a new README with step-by-step instructions for setting up per-tenant settings using the options pattern.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
examples/DotNetCore/WebDemoNet6/WebDemoNet6/Program.cs Corrected spelling in comment “overridden”
examples/DotNetCore/MultiTenantApplicationSetup/README.md Added multi-tenant application setup instructions for .NET
Comments suppressed due to low confidence (1)

examples/DotNetCore/MultiTenantApplicationSetup/README.md:73

  • Before registering ConfigureTenantSettings, add services.AddHttpContextAccessor() to ensure IHttpContextAccessor is available in the DI container.
services.AddScoped<IConfigureOptions<TenantSettings>, ConfigureTenantSettings>();

Comment thread examples/DotNetCore/WebDemoNet6/WebDemoNet6/Program.cs Outdated
Comment thread examples/DotNetCore/MultiTenantApplicationSetup/README.md
Comment thread examples/DotNetCore/MultiTenantApplicationSetup/README.md
@amerjusupovic amerjusupovic merged commit edbeb2b into main May 27, 2025
6 checks passed
@amerjusupovic amerjusupovic deleted the ajusupovic/dotnet-multitenant-info branch May 27, 2025 21:07
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