-
Notifications
You must be signed in to change notification settings - Fork 25k
Update Blazor release notes #36630
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
base: main
Are you sure you want to change the base?
Update Blazor release notes #36630
Conversation
|
The twist is |
|
Does it work on publish output? |
|
Not sure ... I didn't test. I'll check that and see what happens. brb |
|
I see what you mean ... no ... it doesn't fingerprint that file after the app is published. I'm not sure if I was just being silly expecting it to work like that. Anyway ... for the changes that we need here, let me try another pass on this to see if I can get the coverage right. I'm still a bit confused tho because Rubber 🦆 says 'no' to that. I just tried with |
|
I'm attempting to understand this feature better, and I tried the MSBuild config for
Any idea what might be going wrong? |
|
Ah .... rubber 🦆 says, 'that's an |
|
@maraf ... Ok, I think 🤔 that I'm getting this feature now. See if the diff (I made two commits) covers what we need to say. If so, I'll go into the reference article coverage next and make similar updates. |


Fixes #36628
Marek ... I might be incorrect, but it seems like the example that you were referring to is correct by design because it states the module is using the
.jsfile extension ...... and then it goes on to say that if the dev wants to fingerprint
.mjsfiles, they useStaticWebAssetFingerprintPattern.Yes, but that's after the example of a
.jsmodule IF they want to use a different extension (or fingerprint expression). Perhaps, I can fix that lead-in line and make it more specific like this ...... which is on this PR.
However, I might be misunderstanding the feature/problem with what's in the article.