wikibugs should Ignore CC changes
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	valhallasw
	Oct 12 2014, 10:59 AM

Details

	Subject	Repo	Branch	Lines +/-
	Fix wikibugs after Phabricator update	labs/tools/wikibugs2	master	+4 -2

Customize query in gerrit

Related Objects
Search...

View Standalone Graph

This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Status	Assigned	Task
		· · ·
Resolved	yuvipanda	T131 IRC bot to report Phabricator activity exactly like Wikibugs does with Bugzilla
Resolved	Legoktm	T647 wikibugs should Ignore CC changes
Resolved	Legoktm	T75710 Cannot find transaction info for some events
		· · ·

Event Timeline

valhallasw created this task.Oct 12 2014, 10:59 AM

valhallasw raised the priority of this task from to Needs Triage.

valhallasw updated the task description. (Show Details)

valhallasw added a project: Wikibugs.

valhallasw changed Security from none to None.

valhallasw added subscribers: Unknown Object (MLST), valhallasw.

https://phabricator.wikimedia.org/settings/panel/emailpreferences/

Everyone can set their own prefs

• chasemp renamed this task from Ignore CC changes to phawikibugs should Ignore CC changes.Oct 14 2014, 7:18 PM

valhallasw mentioned this in T658: Move 'projects' up in list of Task properties.Oct 15 2014, 9:55 PM

A side-issue, which I assume fixing this Task will solve, but should be checked (or split, or mentioned elsewhere?)...

Currently, if we Comment on a Task that we're not subscribed to, then only the "Subscribed" action is mentioned by phawikibugs,
e.g. here, https://phabricator.wikimedia.org/T792#16443 which was sent to IRC as:
<phawikibugs> Quiddity updated subscribers of T792: UX Testing Environment (Tracking). https://phabricator.wikimedia.org/T792

In T647#16471, @Quiddity wrote:

A side-issue, which I assume fixing this Task will solve, but should be checked (or split, or mentioned elsewhere?)...

T898, I imagine.

In an ideal world, phawikibugs shouldn't bother about comments either, right?

Nemo_bis awarded a token.Oct 29 2014, 7:55 AM

I confirm the requirement. This would be a regression, see https://bugzilla.wikimedia.org/show_bug.cgi?id=15585 (do we have a tag for regressions here?).

In T647#16504, @Qgil wrote:

In an ideal world, phawikibugs shouldn't bother about comments either, right?

It should -- comments are relevant to following what happens, and if possible, phawikibugs should add a snippet of the text to the message on IRC.

Liuxinyu970226 subscribed.Oct 30 2014, 8:45 AM

See T131#17756

Qgil mentioned this in T763: Phabricator far too spammy and off-topic to continue posting to wikibugs-l.Nov 3 2014, 7:18 AM

The new IRC bot does not announce CC-only changes.

Liuxinyu970226 unsubscribed.Nov 6 2014, 12:59 AM

Was this change already pushed live? Because wikibugs2 announced my subscribing to a bug, this morning and yesterday:
<wikibugs2> Needs-Info tag - https://phabricator.wikimedia.org/T1196#20869 (Quiddity)
<wikibugs2> Phabricator.org: Meme button in Phabricator text area bar doesn't work - https://phabricator.wikimedia.org/T906#20599 (Quiddity)

Reopening, it's still happening.

Legoktm renamed this task from phawikibugs should Ignore CC changes to wikibugs should Ignore CC changes.Nov 24 2014, 4:58 AM

Legoktm removed Legoktm as the assignee of this task.

Legoktm added a subtask: T75710: Cannot find transaction info for some events.

Legoktm subscribed.

Betacommand subscribed.Nov 30 2014, 3:28 PM

valhallasw raised the priority of this task from Lowest to High.Nov 30 2014, 3:30 PM

valhallasw moved this task from Backlog to Ready to Go on the Wikibugs board.Nov 30 2014, 3:36 PM

Krinkle subscribed.Dec 2 2014, 5:45 AM

For a task like this, a link to the source code or a documentation page would be helpful. I don't even know where to start. I do know that I'm getting pinged multiple times by wikibugs across channels when subscribing to a task. :-(

https://github.com/wikimedia/labs-tools-wikibugs2

We're waiting on the patch for T75710 to be deployed first, and then I have a patch sitting around locally to fix this.

( Wikibugs could have a link to the project page and/or the repository, although hopefully in a few days the repo can be hosted here with Diffusion...)

Legoktm closed subtask T75710: Cannot find transaction info for some events as Resolved.Dec 10 2014, 7:52 PM