Skip to content

add redirects for deprecated CSP pages#9520

Merged
tanyaparker merged 1 commit into
stagingfrom
csp-redirects
Jul 15, 2016
Merged

add redirects for deprecated CSP pages#9520
tanyaparker merged 1 commit into
stagingfrom
csp-redirects

Conversation

@tanyaparker

Copy link
Copy Markdown
Contributor

No description provided.

@ashercodeorg

Copy link
Copy Markdown
Contributor

Do we have a plan for how long we are going to maintain these redirects?

@@ -0,0 +1 @@
https://curriculum.code.org/csp/unit4v1/

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so v1 is our new naming standard for a while?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Units 4-6 are actually still temporary. I set a reminder to update these again to /csp/unit4 when those pages are launched (in a couple of weeks).

@breville

Copy link
Copy Markdown
Member

lgtm

@tanyaparker

tanyaparker commented Jul 14, 2016

Copy link
Copy Markdown
Contributor Author

@ashercodeorg what is a typical plan for keeping redirects? We have quite a lot that sit in our repo for a long time. Should we base it off traffic in GA, and if it's lower than X pageviews we can remove it?

I would want to eventually remove the unit redirects since those are pages that are replacing the old pages. But sometimes we use the redirect like code.org/widgets as a "short link" that's easier to share and remember. Should we be doing this differently?

@tanyaparker tanyaparker merged commit feb7be0 into staging Jul 15, 2016
@tanyaparker tanyaparker deleted the csp-redirects branch July 15, 2016 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants