Opened 5 years ago

Closed 5 years ago

#223 closed change (fixed)

[trac] Get rid of the in-progress flag

Reported by: fhd Assignee: philll
Priority: P3 Milestone:
Module: Infrastructure Keywords:
Cc: philll, trev Blocked By:
Blocking: Platform:
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/6240833384742912/

Description

Background

We have an "In progress" checkbox now. I don't think a flag is the best way to communicate this: If something is "assigned" it means the assignee will see to it that it gets done. If anyone is wondering if the work already started or how far it is, they can ask the assignee. Plus, this is quite likely to get out of sync with reality, since it'd need to be changed frequently.

What to change

Remove the "In progress" flag.

Change History (8)

comment:1 Changed 5 years ago by fhd

  • Cc philll added

@Philip: I deliberately didn't mark this as "ready", please do so if you consider the discussion on IRC concluded. If not, please voice concerns.

comment:2 Changed 5 years ago by fhd

  • Component changed from Unknown to Infrastructure

comment:3 Changed 5 years ago by philll

I think, it might be a valuable info for our community and especially for our staff, what we are working on right now. I regard this as a part of resource planning.

comment:4 Changed 5 years ago by fhd

  • Cc trev added

Seems we need you here Wladimir, apparently this is still deadlocked.

My point above is pretty much that we have that info anyway, in a way that's not out of sync, if we actually ask. Which is how this is usually handled on public bug trackers from all I can tell.

comment:5 Changed 5 years ago by trev

I agree with fhd, the added value of this flag is negligible. Yet it introduces significant overhead to the work flow as I noticed myself already. It is also something that I tend to forget, we'll have stale or missing flags rather frequently which will negate any potential advantages. To sum up: I don't consider this worth it. It's better to discourage bug hugging instead, issues should only be assigned if they are about to be addressed soon - otherwise they should be free for anybody to take.

comment:6 Changed 5 years ago by philll

  • Owner set to philll
  • Ready set
  • Status changed from new to assigned

comment:7 Changed 5 years ago by philll

  • Review URL(s) modified (diff)
  • Status changed from assigned to reviewing

comment:8 Changed 5 years ago by philll

  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.