Closed
Bug 946993
Opened 12 years ago
Closed 12 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•12 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•12 years ago
|
Whiteboard: [kb=1207234]
Reporter | ||
Comment 2•12 years ago
|
||
Assignee | ||
Comment 3•12 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•12 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•12 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•12 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•12 years ago
|
||
I would just delete these. These banners were before we had the official affiliates.mozilla.org.
Reporter | ||
Comment 8•12 years ago
|
||
Steven - let's go ahead and delete the buttons. Thanks.
Flags: needinfo?(jbertsch)
Reporter | ||
Comment 9•12 years ago
|
||
We can also remove the images in Comment 3, per discussion with Pascal.
Thanks Steven.
Assignee | ||
Comment 10•12 years ago
|
||
Attachment #8343871 -
Flags: review?(pmac)
Comment 11•12 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•12 years ago
|
||
Applied to trunk in r123200.
Whiteboard: [kb=1207234] → [kb=1207234] r=123200
Assignee | ||
Comment 13•12 years ago
|
||
Merged to tags/stage in r123201 and tags/production in r123202. Thanks.
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
•