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

[blazor] Boot handled by dotnet.js #47752

Merged
merged 15 commits into from
Apr 20, 2023
Merged

[blazor] Boot handled by dotnet.js #47752

merged 15 commits into from
Apr 20, 2023

Conversation

maraf
Copy link
Member

@maraf maraf commented Apr 17, 2023

  • Let runtime handle booting
  • Blazor loads boot config only to get dotnet.js URL, before we find a better solution for putting a fingerprint on parts of dotnet.js that change more often
  • This PR doesn't migrate some features, that should be considered common and will be done in later PRs
    • JSInitializers
    • Satelite assemblies
    • Lazy assemblies

@maraf maraf added area-blazor Includes: Blazor, Razor Components feature-blazor-wasm This issue is related to and / or impacts Blazor WebAssembly labels Apr 17, 2023
@maraf maraf self-assigned this Apr 17, 2023
@maraf maraf added this to the 8.0-preview4 milestone Apr 19, 2023
@maraf maraf marked this pull request as ready for review April 20, 2023 07:00
@maraf maraf requested a review from a team as a code owner April 20, 2023 07:00
@maraf maraf requested review from pavelsavara and a team and removed request for a team April 20, 2023 07:00
@@ -19,7 +19,7 @@ public class BootResourceCachingTest
: ServerTestBase<AspNetSiteServerFixture>
{
// The cache name is derived from the application's base href value (in this case, '/')
private const string CacheName = "blazor-resources-/";
private const string CacheName = "dotnet-resources-/";
Copy link
Member

Choose a reason for hiding this comment

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

This will force one time DLL re-download. I think it's fine.

@lewing
Copy link
Member

lewing commented Apr 20, 2023

@javiercn any objections, it would be good to get this in soon

Copy link
Member

@javiercn javiercn left a comment

Choose a reason for hiding this comment

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

Looks good to me, @MackinnonBuck can you take a look at it too?

Copy link
Member

@MackinnonBuck MackinnonBuck left a comment

Choose a reason for hiding this comment

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

Looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components feature-blazor-wasm This issue is related to and / or impacts Blazor WebAssembly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants