Trevor Chadwick

Results 6 comments of Trevor Chadwick

> The unit tests failed on this PR Hey @madskristensen, I was hoping you would have more insight to the right approach here. I'm worried about side-effects of putting the...

> > This proposed change will sanitize all assets to have a leading "/" to ensure this matches. > > Maybe a if statement to .NET 6 will be necessary...

I think the issue is tied to the "TryGetAssetFromRoute" method and the corresponding call to NormalizeRoute in .NET 6. When the asset is added to the pipeline it adds the...

> Any alternative for .NET 6? You can check out my PR for a short-term fix: https://github.com/ligershark/WebOptimizer/pull/240 I think the test just needs to be changed to allow the fix...

I just want to bump this issue, is there anything we can do to help? (I proposed a quick PR fix: #127 )

@logiclabs Mads just merged the code fix for fixing this issue. I hope this means that this will be out with the next release. (I'm not sure what the release...