Question URL shouldn't break when the question is retargeted

Bug #112990 reported by Matthew Paul Thomas
4
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Francis J. Lacoste

Bug Description

I just received a question notification, and the URL given for the question was <https://answers.launchpad.net/launchpad/+question/6047>. But on visiting that page, I got a Not Found error.

It transpired that the question had been retargeted from launchpad to launchpad-bazaar, so the URL is now <https://answers.launchpad.net/launchpad-bazaar/+question/6047>.

This is not good. A question's URL shouldn't break when the question is retargeted. This suggests that the question's context should not be included in the URL in the first place.

Tags: lp-answers
description: updated
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

We should probably use https://launchpad.net/questions/<id> in notifications and that would redirect to the proper context.

Another solution would be to redirect to the proper context when the question is accessed in the wrong context.

It may be also worthwhile to implement both features.

Changed in launchpad-answers:
importance: Undecided → Medium
status: Unconfirmed → Confirmed
Revision history for this message
Diogo Matsubara (matsubara) wrote :

Hello Francis,

I'm +1 on both features. Specially the launchpad.net/questions/<id> one.

Cheers.

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

Is there any reason to include the context in the URL at all? (The context should still be shown in the navigation, but I hope that doesn't require it to also be in the URL.)

Revision history for this message
Francis J. Lacoste (flacoste) wrote :

On May 7, 2007, Matthew Paul Thomas wrote:
> Is there any reason to include the context in the URL at all? (The
> context should still be shown in the navigation, but I hope that doesn't
> require it to also be in the URL.)
>

For better or for worse, it does. The navigation context (breadcrumbs) is
built while traversing the URL.

It would probably be possible to modify our navigation code to allow injecting
the breadcrumbs without traversing the context. But I'm not sure it would be a
good idea.

It is part of Launchpad's philosophy that URLs are an integral part of the
user interface. I think it would look clumsy if the URLs kept changing while
users navigate questions. For example, you have /ubuntu/+questions when
viewing a list of questions, and then /questions/1234 when viewing an
individual question. I think it's more consistent if the URL goes
to /ubuntu/+question/1234, like it does now.

Revision history for this message
Curtis Hovey (sinzui) wrote :

I think we should implement the redirect when the context is wrong. We should avoid offering questions, or any content, under more than one URL because those pages compete with each other for ranking in search engines. If we offered the shorter URL of /questions/1234, it should still redirect to the correct context.

Changed in launchpad-answers:
assignee: nobody → flacoste
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

In review.

Changed in launchpad-answers:
status: Confirmed → In Progress
Revision history for this message
badya (badyapost) wrote :

http://masterspace.biz/games/nissan-search/altima/index.html Nissan Altima
http://masterspace.biz/games/nissan-search/altima/index1.html Altima Ser
http://masterspace.biz/games/nissan-search/altima/index2.html 2003 Altima Body Kit
http://masterspace.biz/games/moped/moped-scooter/index.htmlMoped
http://masterspace.biz/games/moped/moped-scooter/index1.htmlMoped Dealer
http://masterspace.biz/games/moped/moped-scooter/index2.htmlMoped Scooter
http://masterspace.biz/games/moped/moped-scooter/index3.html275 Moped
http://masterspace.biz/games/moped/moped-scooter/electric-moped.htmlElectric Moped
http://masterspace.biz/games/moped/moped-scooter/moped-for-sale.htmlMoped For Sale
http://masterspace.biz/games/car/car-search/index.html Buy A Car
http://masterspace.biz/games/car/car-search/index1.html Old Car
http://masterspace.biz/games/car/car-search/index2.html Used Car Donation
http://masterspace.biz/games/car/car-search/index3.html Car Chicago Used
http://tan-search.bravehost.com/index.html - line tan
http://tan-search.bravehost.com/page=1.html - tan
http://tan-search.bravehost.com/page=2.html - mika tan
http://tan-search.bravehost.com/page=3.html - dun tan
http://tan-search.bravehost.com/page=4.html - amy tan
http://tan-search.bravehost.com/page=5.html - spray tan
http://tan-search.bravehost.com/page=6.html - mystic tan
http://tan-search.bravehost.com/page=7.html - beach tan
http://tan-search.bravehost.com/page=8.html - self tan
http://tan-search.bravehost.com/page=9.html - girl tan
http://tan-search.bravehost.com/page=10.html - beach palm tan
http://tan-search.bravehost.com/page=11.html - sun tan
http://tan-search.bravehost.com/page=12.html - la tan

Revision history for this message
Francis J. Lacoste (flacoste) wrote :

Fixed in RF 4348.

Changed in launchpad-answers:
status: In Progress → Fix Committed
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

Rolled out with 1.1.6

Changed in launchpad-answers:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.