-
Notifications
You must be signed in to change notification settings - Fork 36
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
search is broken on Windows #506
Comments
Hi @zhelyan, thanks for reporting! I've made a potential fix for this issue, but unfortunately, I don't have easy access to a Windows computer to test this. Could you try if this fixes your issue? You can download the binaries containing this fix using the following URL: https://github.com/avisi-cloud/structurizr-site-generatr/actions/runs/9170553411/artifacts/1521911067 |
@dirkgroot sorry for the late reply It is still broken unfortunately. I think the issue is that the method constructs file paths using platform dependant separators. They need to be normalized to http compliant paths. Perhaps the simplest thing to do would be e.g URI(relativeTo + "/" + this).normalize().toURL().toString() |
Another attempt for this issue: #557 |
@zhelyan I've merged the PR since this change is a good idea anyway. Feel free to reopen if this issue still exists. |
Small issue - searching from a component view is broken as relative paths use the windows file separator
\
- i.e the relative path is not a http pathThe text was updated successfully, but these errors were encountered: