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

Uno Hosting #2822

Merged
merged 10 commits into from
Feb 21, 2023
Merged

Uno Hosting #2822

merged 10 commits into from
Feb 21, 2023

Conversation

dansiegel
Copy link
Member

Description of Change

This adds support for using the Uno.Extensions.Hosting's abstractions for the IApplicationBuilder which wraps IHostBuilder. This gives Prism an easy way to get the Window to set the Shell on and makes it easier for developers to customize services that get registered as this uses the same sort of hooks to ensure that both the IHost and Prism use a single container meaning all services registered with IServiceCollection are available with Prism's IContainerProvider and all services registered with Prism's IContainerRegistry will be available from the IHost's IServiceProvider.

Bugs Fixed

  • n/a

API Changes

List all API changes here (or just put None), example:

Added:

  • IHost PrismApplicationBase.Host;
  • IRegionManager PrismApplicationBase.RegionManager
  • void PrismApplicationBase.ConfigureApp(IApplicationBuilder builder);
  • void PrismApplicationBase.ConfigureHost(IHostBuilder builder);
  • void PrismApplicationBase.ConfigureServices(IServiceCollection services);

Changed:

  • public void PrismApplicationBase.Initialize()=> protected void PrismApplicationBase.Initialize(IApplicationBuilder builder);

Removed:

  • protected void InitializeShell(UIElement shell)
    • The purpose of this method was to create the Window and set the Content with the Shell. We now get the Window from the IApplicationBuilder and can easily set the Content and activate the Window as part of the initialization.

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Rebased on top of master at time of PR
  • Changes adhere to coding standard

@dansiegel dansiegel merged commit 1b6bf93 into master Feb 21, 2023
@dansiegel dansiegel deleted the uno-hosting branch February 21, 2023 03:25
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