Last modified: 2013-05-17 08:45:33 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T48937, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 46937 - Missing or useless links in Echo talk page notification e-mails and on wiki
Missing or useless links in Echo talk page notification e-mails and on wiki
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Echo (Other open bugs)
unspecified
All All
: Unprioritized normal with 1 vote (vote)
: ---
Assigned To: Erik Bernhardson
:
: 47954 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-06 01:19 UTC by MZMcBride
Modified: 2013-05-17 08:45 UTC (History)
10 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description MZMcBride 2013-04-06 01:19:07 UTC
Sample message:

---
MediaWiki user Steven (WMF) posted on your talk page:

/* Productization vs. Integration */

View more:

http://www.mediawiki.org/wiki/User_talk:MZMcBride

________________________________________________

To control which emails we send you, visit:
http://www.mediawiki.org/wiki/Special:Preferences#mw-prefsection-echo

Wikimedia Foundation, 149 New Montgomery St., 3rd Fl., San Francisco, CA 94105.

---

Ideally the link would be <https://www.mediawiki.org/wiki/User_talk:MZMcBride#Productization_vs._Integration>.
Comment 1 Nischay Nahata 2013-04-11 15:35:11 UTC
Even the link on the notification popup should also be corrected similarly.
Comment 2 Nischay Nahata 2013-04-11 16:07:16 UTC
Rethinking I figured that the section links are not always there. Sometimes a user just posts without creating a new section. I don't think Echo handles these edits any differently so its not so easily possible and more in the direction of WONTFIX.

Alternatively we can point to the diff as MW's watching emails;for example: http://www.mediawiki.org/w/index.php?title=User_talk:Nischayn22&diff=next&oldid=563821
Comment 3 Nemo 2013-04-26 06:31:23 UTC
(In reply to comment #2)
> Rethinking I figured that the section links are not always there. Sometimes a
> user just posts without creating a new section. I don't think Echo handles
> these edits any differently so its not so easily possible and more in the
> direction of WONTFIX.

Special:Notifications "solves" this by parsing the edit summary so that the automatic edit summary links to the section.

> Alternatively we can point to the diff as MW's watching emails;for example:
> http://www.mediawiki.org/w/index.php?title=User_talk:
> Nischayn22&diff=next&oldid=563821

Yes please.
Comment 4 Nemo 2013-05-02 09:07:40 UTC
(In reply to comment #1)
> Even the link on the notification popup should also be corrected similarly.

That was filed as bug 47954, dunno if it makes sense to separate.
Comment 5 Gerrit Notification Bot 2013-05-14 22:37:41 UTC
Related URL: https://gerrit.wikimedia.org/r/63789 (Gerrit Change I161e2ffda2f2540f64de90cc621fb3b69479d0db)
Comment 6 Ryan Kaldari 2013-05-14 22:46:27 UTC
Is there anything in this bug that is separate from bug 47954? Do we want to change this bug to something like "Add diff links in emails"?
Comment 7 Erik Bernhardson 2013-05-14 22:53:16 UTC
It looks like difflink was already seperated into bug 48183.  As far as i can read this and bug 47954 look to be duplicates of each other, although with the long discussion on bug 47954 i wasn't entirely sure what they were going for.
Comment 8 Risker 2013-05-15 04:15:42 UTC
(In reply to comment #6)
> Is there anything in this bug that is separate from bug 47954? Do we want to
> change this bug to something like "Add diff links in emails"?

Well, the question is whether there needs to be more than one modification here; diffs in the on-wiki notification and a separate request for a diff in the email.  bug 48183 seems to be focused on the former, so this one could be focused on the latter, if it requires two separate changes.
Comment 9 Nemo 2013-05-15 06:03:15 UTC
(In reply to comment #4)
> (In reply to comment #1)
> > Even the link on the notification popup should also be corrected similarly.
> 
> That was filed as bug 47954, dunno if it makes sense to separate.

Apparently not, as the patch is on both.
Comment 10 Nemo 2013-05-15 06:05:13 UTC
*** Bug 47954 has been marked as a duplicate of this bug. ***
Comment 11 Matthew Flaschen 2013-05-15 06:08:24 UTC
Changed title to be specific to talk page, since the rest of the bug already was.
Comment 12 Nischay Nahata 2013-05-17 05:14:16 UTC
The change got merged is this fixed now?
Comment 13 Matthew Flaschen 2013-05-17 08:02:15 UTC
Yes, with that change, it will link to the section if possible, otherwise the whole page.  For diff links, see bug 48183.
Comment 14 Nemo 2013-05-17 08:07:38 UTC
(In reply to comment #13)
> For diff links, see bug 48183.

Is that about email too?
Comment 15 Matthew Flaschen 2013-05-17 08:45:33 UTC
(In reply to comment #14)
> Is that about email too?

Yes, Fabrice specifically mentioned an email diff link. https://bugzilla.wikimedia.org/show_bug.cgi?id=48183#c8

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links