Closed
Bug 946993
Opened 11 years ago
Closed 11 years ago
delete files from SVN (no redirect required)
Categories
(www.mozilla.org :: Legacy PHP system, defect)
www.mozilla.org
Legacy PHP system
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jbertsch, Assigned: sgarrity)
References
Details
(Whiteboard: [kb=1207234] r=123200)
Attachments
(1 file)
166.04 KB,
patch
|
pmac
:
review+
|
Details | Diff | Splinter Review |
Please delete the following files/folders from SVN (no redirect required): http://www.mozilla.com/en-US/firefox/ripcurl http://www.mozilla.com/en-US/firefox/ripcurl/index.html plus more in comment #1
Reporter | ||
Comment 1•11 years ago
|
||
Also for deletion: http://www.mozilla.com/en-US/firefox/install-feedback http://www.mozilla.com/en-US/firefox/install-feedback/index.html http://www.mozilla.com/en-US/firefox/bookmarks http://www.mozilla.com/en-US/firefox/bookmarks/index.html http://www.mozilla.com/en-US/firefox/button.html http://www.mozilla.com/en-US/firefox/buttons http://www.mozilla.com/en-US/firefox/beta/feedback http://www.mozilla.com/en-US/firefox/backtoschool http://www.mozilla.com/en-US/firefox/backtoschool/index.html http://www.mozilla.com/en-US/firefox/beta http://www.mozilla.com/en-US/firefox/beta/faq http://www.mozilla.com/en-US/firefox/beta/features http://www.mozilla.com/en-US/firefox/beta/feedbackprivacypolicy http://www.mozilla.com/en-US/firefox/beta/technology http://www.mozilla.com/en-US/firefox/fb-badges (entire folder - all of it)
Reporter | ||
Updated•11 years ago
|
Whiteboard: [kb=1207234]
Reporter | ||
Comment 2•11 years ago
|
||
Plus http://www.mozilla.com/en-US/firefox/phishing-protection http://www.mozilla.com/en-US/firefox/phishing-protection/index.html
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to Jennifer Bertsch [:jbertsch] from comment #1) > http://www.mozilla.com/en-US/firefox/bookmarks/index.html This bookmarks page is localized in de, en-GB, es-ES, fr, it, pl, pt-BR, and ru. Delete them all?
Assignee: nobody → steven
Flags: needinfo?(jbertsch)
OS: Mac OS X → All
Hardware: x86 → All
Assignee | ||
Comment 4•11 years ago
|
||
The /en-US/firefox/buttons/ dir contains the following images: http://www.mozilla.org/en-US/firefox/buttons/firefox_80x15.png http://www.mozilla.org/en-US/firefox/buttons/firefox_pixel.png http://www.mozilla.org/en-US/firefox/buttons/getfirefox_88x31.png http://www.mozilla.org/en-US/firefox/buttons/getfirefox_large.png http://www.mozilla.org/en-US/firefox/buttons/getfirefox_large2.png http://www.mozilla.org/en-US/firefox/buttons/getfirefox_large3.png http://www.mozilla.org/en-US/firefox/buttons/getfirefox_small.png http://www.mozilla.org/en-US/firefox/buttons/header.png http://www.mozilla.org/en-US/firefox/buttons/takebacktheweb_large.png http://www.mozilla.org/en-US/firefox/buttons/takebacktheweb_small.png These are obviously out of date (several logo-iterations old). That said, they are probably still sprinkled around the web as links in old blog sidebars, etc. Just wanted to confirm that we should delete them. The only really meaningful way to update them would be to create new images at each of the dimensions, which probably isn't worth it.
Assignee | ||
Comment 5•11 years ago
|
||
Pascal, when we're removing http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/en-US/firefox/fb-badges/ can the SVN external locale http://svn.mozilla.org/projects/l10n-misc/trunk/badges/locale/ go too?
Flags: needinfo?(pascalc)
Comment 6•11 years ago
|
||
Loioking at the content of the strings, this is all Firefox 4 release party related, I don't remember exactly working on that part of the site but I think it's safe to remove the svn external.
Flags: needinfo?(pascalc)
Comment 7•11 years ago
|
||
I would just delete these. These banners were before we had the official affiliates.mozilla.org.
Reporter | ||
Comment 8•11 years ago
|
||
Steven - let's go ahead and delete the buttons. Thanks.
Flags: needinfo?(jbertsch)
Reporter | ||
Comment 9•11 years ago
|
||
We can also remove the images in Comment 3, per discussion with Pascal. Thanks Steven.
Assignee | ||
Comment 10•11 years ago
|
||
Attachment #8343871 -
Flags: review?(pmac)
Comment 11•11 years ago
|
||
Comment on attachment 8343871 [details] [diff] [review] Patch to remove files from SVN Review of attachment 8343871 [details] [diff] [review]: ----------------------------------------------------------------- LGTM. Thanks :sgarrity!
Attachment #8343871 -
Flags: review?(pmac) → review+
Assignee | ||
Comment 12•11 years ago
|
||
Applied to trunk in r123200.
Whiteboard: [kb=1207234] → [kb=1207234] r=123200
Assignee | ||
Comment 13•11 years ago
|
||
Merged to tags/stage in r123201 and tags/production in r123202. Thanks.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•