-
Notifications
You must be signed in to change notification settings - Fork 45
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
Specific case with redirect for trailing locale #615
Conversation
ilc/server/tailor/fragment-hooks.js
Outdated
@@ -46,8 +46,8 @@ function insertStart(logger, stream, attributes, headers) { | |||
isAsync | |||
? `<!-- Async fragments are not fully implemented yet: ${uri} -->` | |||
: id | |||
? asyncStylesLoadTemplate(uri, id) | |||
: '', | |||
? asyncStylesLoadTemplate(uri, id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to update node_module and then run "npm run format"
ilc/common/UrlProcessor.js
Outdated
@@ -29,29 +29,27 @@ class UrlProcessor { | |||
|
|||
#processUrlTrailingSlash = (url) => { | |||
if (!UrlProcessor.validSchemes.some((scheme) => url.startsWith(scheme))) { | |||
// any combination of slashes at the beginning URL class classifies as an absolute link with http: protocol, so we remove them at the end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this comment is useful actually, bring it back
Coverage ReportIlc/serverCommit SHA:c309aa5f1701100b59e122952c727a5cb8bba1bd Test coverage results 🧪
File details
Ilc/clientCommit SHA:c309aa5f1701100b59e122952c727a5cb8bba1bd Test coverage results 🧪
File details
RegistryCommit SHA:c309aa5f1701100b59e122952c727a5cb8bba1bd Test coverage results 🧪
File details
|
No description provided.