unoplatform / Uno.Wasm.Bootstrap

A simple nuget package to run C# code in a WASM-compatible browser
Other
364 stars 57 forks source link

The PWA manifest link should be at the root #872

Closed jeromelaban closed 1 month ago

jeromelaban commented 1 month ago

GenerateIndexHtml invalidates the modifications done by GeneratePWAContent:

Possible solution:

html = html.Replace($"\"./", $"\"{WebAppBasePath}{_remoteBasePackagePath}/");
html = html.Replace($"\"{WebAppBasePath}{_remoteBasePackagePath}/{Path.GetFileName(PWAManifestFile)}", $"{WebAppBasePath}{Path.GetFileName(PWAManifestFile)}");

The PWA manifest must always be at the root.

jeromelaban commented 1 month ago

Fixed in https://github.com/unoplatform/Uno.Wasm.Bootstrap/pull/873