All users were logged out of Bugzilla on October 13th, 2018

Redirect /kb/Firefox+Support+Home+Page and other special pages

RESOLVED FIXED in 1.5.7

Status

P2
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: cww, Assigned: jsocol)

Tracking

unspecified
1.5.7
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

8 years ago
We should make sure special pages are redirected rather than returning 404s
(Assignee)

Updated

8 years ago
Assignee: nobody → james
OS: Mac OS X → All
Priority: -- → P2
Hardware: x86 → All
(Assignee)

Comment 1

8 years ago
Besides /kb/Firefox Support Home Page, what other special pages, and to what targets?
(Reporter)

Comment 2

8 years ago
Firefox Help
Windows start page
Firefox Support Home Page
> /home
Localization Dashboard > /contributors
Live Chat > /chat
Mobile Help and Tutorials /mobile
Support Website Forums > /en-US/questions
(Assignee)

Comment 3

8 years ago
Easy enough. I'm going to let this sit until probably Tuesday in case any others come up.

Comment 4

8 years ago
Translating content blocks
Dynamic Content
All Knowledge Base articles

Comment 5

8 years ago
"Ask a question" is another important one. Cww, did we decide to create a wiki page for that on Kitsune, or a special page?
(Assignee)

Comment 6

8 years ago
(In reply to comment #4)
> Translating content blocks
> Dynamic Content
> All Knowledge Base articles

Two of these don't exist anymore. Where should they point?

(In reply to comment #5)
> "Ask a question" is another important one. Cww, did we decide to create a wiki
> page for that on Kitsune, or a special page?

IIRC wiki page, but I'm not 100% sure.

Comment 7

8 years ago
Michael says the same thing, so I think you recall correctly.

Comment 8

8 years ago
(In reply to comment 6)
> Two of these don't exist anymore. Where should they point?
These 3 pages don't exist (error 404) :
* "Translating content blocks" and "Dynamic content" must point to a section about templates in contributor home page. But I don't know where and I don't think that every "how to contribute" pages point somewhere. There is nothing about that in the mockup.
* "All Knowledge Base articles" is equivalent to the localization dashboard. It is useless in Kitsune.

(In reply to comment 5)
> "Ask a question" is another important one.
This page exists (no error 404). But it points no where. It must point to "Ask a question" button of the home page.
(Assignee)

Comment 9

8 years ago
(In reply to comment #8)
> * "Translating content blocks" and "Dynamic content" must point to a section
> about templates in contributor home page.

If these are/were wiki documents, maybe we can just create them as redirects later? Without a specific target URL I can't do this in .htaccess

> * "All Knowledge Base articles" is equivalent to the localization dashboard. It
> is useless in Kitsune.

We also have /kb/all.

Comment 10

8 years ago
Kadir,
Do you know where each "How to contribute" page must point?
I see Toolbox and Contributing section in Contributor Dashboard mockup that are designed for that. But what pages in which section?
(Reporter)

Comment 11

8 years ago
I deliberately left the wiki documents out of this list for that reason.  Ditto the stuff linked from the How to contribute pages.  We need to write a bunch of new pages that will be the targets of other pages but none of that is relevant to this bug.

Let's point All Knowledge Base articles to /all since it was previously linked from the start page and it may have been bookmarked/crawled.
And let's rename Ask a question to just /ask once we've migrated it (we can do this manually though). 

And, make sure all links to that page are indeed pointing to /ask from now on. For example, at the bottom of each article today, we're instead linking straight to the forum's new question form. This might be a separate bug though -- adding the comment here in case it isn't.
(Assignee)

Comment 13

8 years ago
(In reply to comment #12)
> And let's rename Ask a question to just /ask once we've migrated it (we can do
> this manually though). 

To be clear, if it's a wiki document, the URL has to be /kb/ask.
Right. I meant that. :) Thanks
(Assignee)

Comment 15

8 years ago
Mind adding that to the update-link-target etherpad for bug 611013?
Will do.
(Reporter)

Comment 17

8 years ago
Get+help+with+Firefox+4+Beta > /home (we'll use showfor to make it show beta content).
(In reply to comment #17)
> Get+help+with+Firefox+4+Beta > /home (we'll use showfor to make it show beta
> content).

<offtopic>Cheng, can you work on getting some test content up on /home to verify that this will be sufficient for our needs?</offtopic>

Comment 19

8 years ago
In reply to comment 11:
> Let's point All Knowledge Base articles to /all since it was previously linked
> from the start page and it may have been bookmarked/crawled.
Now, "All Knowledge Base articles" points to the expanded dashboard. So:
All Knowledge Base articles > /contributors


Article list > /all (i.e. /kb/all filtered for Troubleshooting and Help categories)
(Assignee)

Comment 20

8 years ago
This is the list I'm planning on putting up for review. Any final additions/changes? ($1 is the incoming locale.)

# Redirect old KB docs to new locations
/kb/Firefox+Help                  /$1/home/
/kb/Windows+start+page            /$1/home/
/kb/Firefox+Support+Home+Page     /$1/home/
/kb/Get+help+with+Firefox+4+Beta  /$1/home/
/kb/Localization+Dashboard        /$1/contributors
/kb/Live+Chat                     /$1/chat/
/kb/Mobile+Help+and+Tutorials     /$1/mobile/
/kb/Support+Website+Forums        /en-US/questions/
/kb/All+Knowledge+Base+articles   /$1/kb/all
Summary: Redirect https://master.support.mozilla.com/en-US/kb/Firefox+Support+Home+Page and other special pages → Redirect /kb/Firefox+Support+Home+Page and other special pages
The list looks good to me. Just one disclaimer about the use of the word "final": if we do spot more things, we'll still let you know about it. ;)

But I think this is pretty much it. Thanks!
(Assignee)

Comment 22

8 years ago
Created attachment 491216 [details] [diff] [review]
htaccess changes

Doing these in htaccess to get in ahead of the s/+/ / and locale middleware.
Attachment #491216 - Flags: review?(paulc)

Comment 23

8 years ago
Two errors, one missing:
/kb/Live+Chat                     /en-US/chat/
/kb/All+Knowledge+Base+articles   /$1/contributors
/kb/Article+list                  /$1/kb/all
(Assignee)

Comment 24

8 years ago
Created attachment 491230 [details] [diff] [review]
moar htaccess changes
Attachment #491216 - Attachment is obsolete: true
Attachment #491216 - Flags: review?(paulc)
(Assignee)

Updated

8 years ago
Attachment #491230 - Flags: review?(paulc)
Comment on attachment 491230 [details] [diff] [review]
moar htaccess changes

My only issue with this is that it doesn't seem to work with spaces instead of pluses. E.g. /kb/Localization Dashboard doesn't work but /kb/Localization+Dashboard does.
(Assignee)

Comment 26

8 years ago
Tiki didn't generate links with spaces. This is to capture bookmarks and other pre-existing inbound links, which should all be pluses.
Ah, one moar thing: some rules still redirect to Get Help with Firefox 4 beta?as=u
Do we want to update those as well?
(Assignee)

Comment 28

8 years ago
(In reply to comment #27)
> Do we want to update those as well?

Yes. On it.
(Assignee)

Comment 29

8 years ago
Created attachment 491349 [details] [diff] [review]
yarly! even moar htaccess changes!

I don't know why that rule was in there twice...
Attachment #491230 - Attachment is obsolete: true
Attachment #491349 - Flags: review?(paulc)
Attachment #491230 - Flags: review?(paulc)
Comment on attachment 491349 [details] [diff] [review]
yarly! even moar htaccess changes!

WFM, with the mention that those 2 old beta rewrites now go to the home page.
Attachment #491349 - Flags: review?(paulc) → review+
(Assignee)

Comment 31

8 years ago
r77719.

Also putting it in 1.5.7 with all the other SVN/.htaccess stuff.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: 2.3 → 1.5.7
(Assignee)

Updated

8 years ago
Flags: in-testsuite?
Looks like you forgot to put backslash here:
 RewriteRule ^kb/Article+list$ /kb/all [L,NC,R=301]

Reopening.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Flags: in-testsuite? → in-testsuite+
(Assignee)

Comment 33

8 years ago
r77926.
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago8 years ago
Resolution: --- → FIXED
/Article+list should be localized.
Reopening.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 35

8 years ago
r77930.

Hopefully that's the last thing we've missed.
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.