Last modified: 2013-02-15 16:30:12 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 T37532, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 35532 - Include unified diff in gerrit email notifications
Include unified diff in gerrit email notifications
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Git/Gerrit (Other open bugs)
unspecified
All All
: Low normal (vote)
: ---
Assigned To: Chad H.
: upstream
Depends on: 39589
Blocks: 22596
  Show dependency tree / graph
 
Reported: 2012-03-27 19:38 UTC by Nemo
Modified: 2013-02-15 16:30 UTC (History)
4 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Nemo 2012-03-27 19:38:41 UTC 
Because the notifications are currently useless, they should be as they were in the system (example: http://thread.gmane.org/gmane.org.wikimedia.mediawiki.cvs/112867 ).
Comment 1 Chad H. 2012-06-14 01:36:18 UTC 
Checked on this upstream. Turns out it's already been fixed in master[0]. Will be released as part of 2.5, so marking this LATER until 2.5 is out and we determine a timeline for upgrading.

[0] https://gerrit-review.googlesource.com/#/c/34861/
Comment 2 Nemo 2012-11-14 12:16:20 UTC 
Switching from LATER to the second most relevant resolution for fear of information loss. http://article.gmane.org/gmane.science.linguistics.wikipedia.technical/65116
Comment 3 Chad H. 2013-02-12 15:40:14 UTC 
There should now be diffs in your e-mails for new changes.
Comment 4 Chad H. 2013-02-12 15:41:14 UTC 
Hmm, not there. Need to poke this further (they *are* available)
Comment 5 Chad H. 2013-02-15 16:30:12 UTC 
We've now got diffs, Gerrit change #49133.
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links