Skip to content

Add Web Worker sample app#588

Merged
guardrex merged 3 commits intomainfrom
guardrex/add-webworkers-sample
Oct 3, 2025
Merged

Add Web Worker sample app#588
guardrex merged 3 commits intomainfrom
guardrex/add-webworkers-sample

Conversation

@guardrex
Copy link
Contributor

@guardrex guardrex commented Oct 2, 2025

Fixes #587
Addresses dotnet/AspNetCore.Docs#36168

Article PR: dotnet/AspNetCore.Docs#36175

Ilona ... best for you to take a quick look 👁️ at the sample app placement and edited README.

Also, see if "Web Worker" (singular) is best for this. We're using "Web Workers" generally in the article.

@guardrex guardrex requested a review from ilonatommy October 2, 2025 16:35
@guardrex guardrex changed the title Add Web Workers sample app Add Web Worker sample app Oct 2, 2025
@ilonatommy
Copy link
Member

Also, see if "Web Worker" (singular) is best for this. We're using "Web Workers" generally in the article.

I agree, in the common web development terminology it's plural. The demo implementation works with the concept/API of Web Workers, even if implementing a single worker and we don't want to specially underline the fact that we're using only one. Proposed change to "Workers" looks like a good generalization.

@guardrex

This comment was marked as resolved.

@ilonatommy
Copy link
Member

One more ❓ ... Do you want to shed the live demo link because you wouldn't need to maintain this sample? I suspect that it will eventually go stale sometime in the future.

Do you mean removing the "https://ashy-island-008420303.4.azurestaticapps.net/" link? Yes, in case my azure resources get removed, it will die.

@guardrex guardrex merged commit e7265d0 into main Oct 3, 2025
1 check passed
@guardrex guardrex deleted the guardrex/add-webworkers-sample branch October 3, 2025 14:02
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.

Add Web Workers sample app

2 participants