Skip to content

[Blazor] Move Service worker to static web assets in preparation for compressing and fingerprinting everything #39177

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 4, 2024
Merged

[Blazor] Move Service worker to static web assets in preparation for compressing and fingerprinting everything #39177

merged 1 commit into from
Mar 4, 2024

Conversation

javiercn
Copy link
Member

@javiercn javiercn commented Mar 4, 2024

  • Copied the task and targets to static web assets.
  • Changed namespace
  • Left original task in BlazorWasm SDK for 5.0
  • Fixed tests

@ghost ghost added Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch untriaged Request triage from a team member labels Mar 4, 2024
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Thanks for your PR, @javiercn.
To learn about the PR process and branching schedule of this repo, please take a look at the SDK PR Guide.

@javiercn javiercn marked this pull request as ready for review March 4, 2024 16:52
@javiercn javiercn requested a review from a team as a code owner March 4, 2024 16:52
@javiercn javiercn merged commit 77911cb into main Mar 4, 2024
@javiercn javiercn deleted the javiercn/static-web-assets-service-worker branch March 4, 2024 19:07
@javiercn javiercn mentioned this pull request Jun 2, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MackinnonBuck MackinnonBuck MackinnonBuck approved these changes

Assignees
No one assigned
Labels
Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch untriaged Request triage from a team member
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@javiercn @MackinnonBuck