Closed
Bug 885856
Opened 12 years ago
Closed 12 years ago
Handle /projects/index.* files
Categories
(www.mozilla.org :: Legacy PHP system, defect)
www.mozilla.org
Legacy PHP system
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: hoosteeno, Assigned: sgarrity)
References
(Blocks 1 open bug)
Details
(Whiteboard: [kb=1093831] )
Attachments
(1 file)
|
111.20 KB,
patch
|
pmac
:
review+
|
Details | Diff | Splinter Review |
There are a handful of index.*.html files here:
http://viewvc.svn.mozilla.org/vc/projects/mozilla.org/trunk/projects/
They are still served:
http://www.mozilla.org/projects/index.html
http://www.mozilla.org/projects/index.fr.html
http://www.mozilla.org/projects/index.hr.html
http://www.mozilla.org/projects/index.sq.html
http://www.mozilla.org/projects/index.de.html
This bug's overall intention is to get those files out of SVN. Please...
* Determine if they need to be migrated
* If so, determine where
* Migrate them
* Set up necessary redirects
* Delete them from SVN
* If they are the last thing in the about folder, also delete about from SVN
| Reporter | ||
Comment 1•12 years ago
|
||
The last point above should read:
* If they are the last thing in the PROJECTS folder, also delete PROJECTS from SVN
| Reporter | ||
Comment 2•12 years ago
|
||
David, since /projects itself is already redirected, do we need to do anything besides delete index.html?
Flags: needinfo?(dboswell)
| Reporter | ||
Comment 3•12 years ago
|
||
Pascal, can you advise on the localized pages?
Flags: needinfo?(pascalc)
Comment 4•12 years ago
|
||
(In reply to Justin Crawford [:hoosteeno] from comment #2)
> David, since /projects itself is already redirected, do we need to do
> anything besides delete index.html?
Sure, I have no problem with removing that old projects index page. Since projects/index.html isn't redirecting now though, will it redirect when that page is removed?
Flags: needinfo?(dboswell)
Updated•12 years ago
|
Blocks: mozorg-redirects
| Reporter | ||
Comment 5•12 years ago
|
||
OK, so to close this bug we need to...
1) Redirect requests for http://www.mozilla.org/projects/index.html to http://www.mozilla.org/en-US/products/. Do this somewhere around https://github.com/mozilla/bedrock/blob/master/etc/httpd/global.conf#L131
2) Delete these index files from SVN
| Reporter | ||
Updated•12 years ago
|
Whiteboard: [kb=1093831]
| Assignee | ||
Comment 6•12 years ago
|
||
(In reply to Justin Crawford [:hoosteeno] from comment #5)
> 1) Redirect requests for http://www.mozilla.org/projects/index.html to
> http://www.mozilla.org/en-US/products/. Do this somewhere around
> https://github.com/mozilla/bedrock/blob/master/etc/httpd/global.conf#L131
The English index.html is already redirecting, as now any x/index.html is now treated as just x/.
I have a PR for the other locales here: https://github.com/mozilla/bedrock/pull/1280
Comment 7•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/bedrock
https://github.com/mozilla/bedrock/commit/4429d432795cb237d15e95c7fbd5bdbb24298f9b
Redirect locale-specific proect pages (Bug 885856)
https://github.com/mozilla/bedrock/commit/04139ccfab88eeb63e8ced3564e307afa5ef0663
Merge pull request #1280 from sgarrity/bug-885856-projects-index
Redirect locale-specific proect pages (Bug 885856)
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → steven
| Assignee | ||
Comment 8•12 years ago
|
||
Here's a simple delete-only patch. This removes the index* files and a few other .html files that have been previously ported to bedrock.
Attachment #812171 -
Flags: review?(pmac)
Comment 9•12 years ago
|
||
Comment on attachment 812171 [details] [diff] [review]
bug-885856.diff
Review of attachment 812171 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good to me. We need to wait until that recent rewrite rule is deployed to prod before pushing this out though right?
Attachment #812171 -
Flags: review?(pmac) → review+
Comment 10•12 years ago
|
||
https://www.allizom.org/fr/products/
https://www.allizom.org/hr/products/
https://www.allizom.org/sq/products/
https://www.allizom.org/de/products/
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 11•12 years ago
|
||
verified fixed
https://www.mozilla.org/fr/products/
https://www.mozilla.org/hr/products/
https://www.mozilla.org/sq/products/
https://www.mozilla.org/de/products/
Status: RESOLVED → VERIFIED
| Assignee | ||
Comment 12•12 years ago
|
||
Flags: needinfo?(pascalc)
You need to log in
before you can comment on or make changes to this bug.
Description
•