Closed
Bug 960551
Opened 12 years ago
Closed 12 years ago
Cleanup remaining files in /en-US/firefox/fb/
Categories
(www.mozilla.org :: Legacy PHP system, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sgarrity, Assigned: sgarrity)
References
Details
(Whiteboard: [kb=1264063] )
Attachments
(1 file)
|
25.67 KB,
patch
|
pmac
:
review+
|
Details | Diff | Splinter Review |
We have the following Facebook pages still in the PHP-based SVN repository:
https://www.mozilla.org/en-US/firefox/fb/download-aurora/
Still appears to be in use at: https://www.facebook.com/FirefoxAurora/app_141615939274509 We should replace this with a more modern page on Bedrock if it still required.
https://www.mozilla.org/en-US/firefox/fb/download-beta/
Broken? Doesn't appear to be in use on https://www.facebook.com/FirefoxBeta
https://www.mozilla.org/en-US/firefox/fb/welcome/
https://www.mozilla.org/en-US/firefox/fb/welcome-aurora/
https://www.mozilla.org/en-US/firefox/fb/welcome-channels/
Are any of these used?
| Assignee | ||
Comment 1•12 years ago
|
||
jbertsch: can you answer these, or CC the person who is responsible for the pages on Facebook? Thanks.
Flags: needinfo?(jbertsch)
Comment 2•12 years ago
|
||
Hi Laura and Grace-
Do we need any of these pages or can they be deleted?
Thx,
Jen
Flags: needinfo?(lforrest)
Flags: needinfo?(jbertsch)
Flags: needinfo?(gjimenez)
Comment 3•12 years ago
|
||
This is a good question for Michaela who manages our Facebook account. I'm not sure if these are still in use or not. Michaela?
Flags: needinfo?(lforrest)
Comment 4•12 years ago
|
||
Can be deleted. Thanks!
| Assignee | ||
Comment 5•12 years ago
|
||
(In reply to mthayer@mozilla.com from comment #4)
> Can be deleted. Thanks!
Thanks! Just to confirm, are you sure that https://www.mozilla.org/en-US/firefox/fb/download-aurora/
can be deleted? It looks like it might be used for https://www.facebook.com/FirefoxAurora/app_141615939274509
Flags: needinfo?(mthayer)
Updated•12 years ago
|
Flags: needinfo?(mthayer)
Comment 6•12 years ago
|
||
Ah, thanks Steven. Adding William R. Do you know who manages the Firefox Aurora Facebook page?
My recommendation would be to delete the tab anyways because it's not mobile compatible which is where most people use Facebook. But I don't manage that page.
Comment 7•12 years ago
|
||
I'm fine with deleting the Aurora download page on Facebook if WilliamR signs off.
Flags: needinfo?(gjimenez) → needinfo?(williamr)
Comment 8•12 years ago
|
||
I think it's alright to delete the Aurora download page on Facebook. The Facebook page for Firefox Aurora has not been updated in over a year and is not actively maintained.
Flags: needinfo?(williamr)
| Assignee | ||
Comment 9•12 years ago
|
||
williamr has removed the page now too (thanks!) I'll submit a patch to remove the leftover files in SVN.
| Assignee | ||
Comment 10•12 years ago
|
||
Now that none of these pages are used on Facebook anymore, we can remove them all from SVN:
D en-US/firefox/fb
D en-US/firefox/fb/css
D en-US/firefox/fb/css/download-common.css.php
D en-US/firefox/fb/download-aurora
D en-US/firefox/fb/download-aurora/images
D en-US/firefox/fb/download-aurora/images/bg.jpg
D en-US/firefox/fb/download-aurora/images/title.png
D en-US/firefox/fb/download-aurora/index.html
D en-US/firefox/fb/download-beta
D en-US/firefox/fb/download-beta/images
D en-US/firefox/fb/download-beta/images/bg.png
D en-US/firefox/fb/download-beta/images/title.png
D en-US/firefox/fb/download-beta/index.html
D en-US/firefox/fb/welcome
D en-US/firefox/fb/welcome-aurora
D en-US/firefox/fb/welcome-aurora/images
D en-US/firefox/fb/welcome-aurora/images/bg.jpg
D en-US/firefox/fb/welcome-aurora/images/like.png
D en-US/firefox/fb/welcome-aurora/index.html
D en-US/firefox/fb/welcome-channels
D en-US/firefox/fb/welcome-channels/images
D en-US/firefox/fb/welcome-channels/images/bg.jpg
D en-US/firefox/fb/welcome-channels/index.html
D en-US/firefox/fb/welcome/images
D en-US/firefox/fb/welcome/images/bg.jpg
D en-US/firefox/fb/welcome/index.html
Attachment #8371099 -
Flags: review?(pmac)
| Assignee | ||
Updated•12 years ago
|
Whiteboard: [kb=1264063]
Comment 11•12 years ago
|
||
Comment on attachment 8371099 [details] [diff] [review]
Patch to remove /firefox/fb/ files from SVN
Review of attachment 8371099 [details] [diff] [review]:
-----------------------------------------------------------------
It all looks old and removable to me. This bug deals with the Aurora stuff specifically though, and it appears that there is some beta stuff in this patch as well. Is this intended or collateral damage? It looks fine, so I'll r+, but just double check the beta stuff before merging if you would.
r+wc
Attachment #8371099 -
Flags: review?(pmac) → review+
| Assignee | ||
Comment 12•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•