fix(v2): redirect from should work with trailingSlash: true #5093
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
Follow-up of #5055 (comment)
When using
trailingSlash: true
, the redirect plugin does not emit files forfromExtensions: ['html']
because it ignorers all paths ending with/
If there's a path such as
/xyz/
, we should emit/xyz.html/index.html
, otherwise deployments like Netlify wouldn't redirect when browsing/xyz.html
, they would just serve the HTML file and it would lead to a 404 error due to the route not existing on the frontend router.Also cleanup now useless warning since we reverted to emit
/xyz.html/index.html
, as the warning does not make sense anymoreHave you read the Contributing Guidelines on pull requests?
yes
Test Plan
test + preview
This URL now works fine, while it used to display a frontend 404 error: https://deploy-preview-5093--docusaurus-2.netlify.app/docs/typescript-support.html
Related PRs
#5085