adding redirects so that old website API links surfaced from searches #16288
Conversation
… redirect to proper locations in new website
docs/static_site/src/.htaccess
Outdated
| Redirect 301 /tutorials/index.html /api | ||
| Redirect 301 /architecture/index.html /api/architecture/overview | ||
| Redirect 301 /community/ecosystem.html /ecosystem | ||
| Redirect 301 community/powered_by.html /ecosystem |
|
@aaronmarkham @marcoabreu, please merge asap if CI passes 👍 this will be very helpful to our users, and if not, please give the CI a little nudge, thanks! |
There was a problem hiding this comment.
I tested this on beta and it breaks the site with a 500 error.
https://github.com/apache/incubator-mxnet-site/blob/beta-site/.htaccess
https://mxnet-beta.staged.apache.org/
|
@aaronmarkham beta doesn't work with current htaccess prod config either, I think it's a bug of beta env, it works on dev http://54.211.76.74/tutorials/index.html |
aaronmarkham
dismissed
their stale review
the problem was the staging server, not this PR
aaronmarkham
changed the title
|
@sad- I thought changing the name of the PR would trigger CI - it has in the past. Anyways, when #16304 gets merged, please rebase this PR, so it'll pass CI. |
… redirect to proper locations in new website
sojiadeshina
force-pushed
the
website-link-fixes
branch
from
768e9cf
to
50557a8
Compare
Refer to https://issues.apache.org/jira/browse/INFRA-19187 - this should fix the staging site.
|
closing and reopening under my own fork because current hypothesis is that "sad-" breaks the CI somewhere in bash |
… redirect to proper locations in new website
Description
(Brief description on what this PR is about)
This allows the links in the top results of web searches with queries like "mxnet *** api" redirect to the right location in the new website. working on redirects for search results with links that refer to a specific version as well.