Closed
Bug 545278
Opened 15 years ago
Closed 15 years ago
forum URLs under 'My Watches' are not pretty
Categories
(support.mozilla.org :: Mobile, task)
support.mozilla.org
Mobile
Tracking
(Not tracked)
VERIFIED
FIXED
1.5.2
People
(Reporter: vish_moz, Assigned: paulc)
References
()
Details
Attachments
(1 file)
|
2.00 KB,
patch
|
jsocol
:
review+
|
Details | Diff | Splinter Review |
forum URLs under 'My Watches' are not pretty
STR:
1. log into SUMO mobile stag
2. go to 'My Watches' https://mobile.support-stage.mozilla.org/tiki-user_watches.php?locale=en-US
Actual result:
the forum URLs are not pretty, for eg. https://mobile.support-stage.mozilla.org/tiki-view_forum_thread.php?locale=en-US&forumId=5&comments_parentId=5
Expected result:
forum URLs should be in pretty format like https://mobile.support-stage.mozilla.org/en-US/forum/5/5
Comment 1•15 years ago
|
||
We patched this on trunk. Paul, I'm >90% sure you did the original patch, think you can dig it up?
Assignee: nobody → paulc
Hardware: x86 → All
Target Milestone: --- → 1.5.2
| Assignee | ||
Comment 2•15 years ago
|
||
Actually, I don't see these being friendly on trunk:
https://support.mozilla.com/tiki-user_watches.php
Comment 3•15 years ago
|
||
Not all are, but new ones on trunk should be. Try desktop staging.
| Assignee | ||
Comment 4•15 years ago
|
||
From bug 522151 -- applied the patch, and here are the missing bits. The rest are already on the fennec branch.
I'm already seeing friendly URLs in "My watches" after going to:
https://mobile.support-stage.mozilla.org/en-US/forum/5/5?watch_event=forum_post_thread&watch_object=5&watch_action=add
Attachment #426708 -
Flags: review?(james)
Comment 5•15 years ago
|
||
Comment on attachment 426708 [details] [diff] [review]
missing watch url part
Sorry it took me so long to get to this.
Attachment #426708 -
Flags: review?(james) → review+
| Assignee | ||
Comment 6•15 years ago
|
||
r63166 (on fennec branch, of course)
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
| Reporter | ||
Comment 7•15 years ago
|
||
Verified, FIXED.
Filed Bug 548366 related to this one.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•