-
-
Notifications
You must be signed in to change notification settings - Fork 116
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
Redirect old urls to the corresponding page in the new viewer #184
Comments
the rewrite engine should probably be fed a map based on 2.12's docs since some urls will have to land at ember and others at ember-data. |
We possibly also need a strategy for redirecting |
@sivakumar-kailasam Confirmed on the mapping. My impression as I look at our url structure though is that we may not need a complete mapping, but may be able to handle a simpler set of wildcard rules that handle things. |
Also opened a new issue for us to track info on @toddjordan's comment above |
Supporting legacy module URLs: #242 |
not quite how written up, but /api is live now 🙃 |
PRs for this issue can be explored simultaneously with #185 and #216, but the solution for this is blocked until we have clarity on both:
/current
url to redirect all old urls to Add a /current route that we can use in various places #185Before we shift out of
beta
and replace the main api docs pages on the Ember website, we need to ensure that we've properly remapped all old api urls to their corresponding pages in the new Ember app. Some of ournamespaces
(as shown on the old sidebar) actually have the wordclasses
in their url, and we'll need to account for thatExample urls that we'll want to redirect:
/api/ember/current/modules/ember-application
/api/ember/current/namespaces/Ember.computed
/api/ember/current/namespaces/Ember.computed
To do:
/api
instead ofapi-new
(https://github.com/emberjs/website/blob/master/static.json#L4)emberjs/website
for the above changes with a "DON'T MERGE" title (as we want to make sure this happens when desired and not because someone merges before we're ready)The text was updated successfully, but these errors were encountered: