Skip to content

Redirect to archived devpreview release notes#1285

Closed
sgarrity wants to merge 1 commit intomozilla:masterfrom
sgarrity:bug-858315-devpreview-notes
Closed

Redirect to archived devpreview release notes#1285
sgarrity wants to merge 1 commit intomozilla:masterfrom
sgarrity:bug-858315-devpreview-notes

Conversation

@sgarrity
Copy link
Contributor

@sgarrity sgarrity commented Oct 1, 2013

Bug 858315

Copy link
Contributor

Choose a reason for hiding this comment

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

This block of rules can be simplified as:

RewriteRule ^/projects/devpreview/firstrun(?:/(?:index.html)?)?$ /firefox/firstrun/ [L,R=301] 
RewriteRule ^/projects/devpreview/(.*)$ http://website-archive.mozilla.org/www.mozilla.org/devpreview_releasenotes/projects/devpreview/$1 [L,R=301]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated, thanks.

Copy link
Contributor

Choose a reason for hiding this comment

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

It needs to be exactly what I put. The firstrun rule has to come first or the other will be the only one ever used.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@pmclanahan
Copy link
Contributor

Fixed commit message and merged to master as c814735.

Thanks! 🍰

@pmclanahan pmclanahan closed this Oct 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants