Edgewall Software
Modify

Ticket #1547 (closed enhancement: wontfix)

Opened 7 years ago

Last modified 5 years ago

Define QA owner for a component

Reported by: pkou at ua.fm Owned by: cmlenz
Priority: normal Milestone: 0.9
Component: admin/console Version: 0.8
Severity: normal Keywords:
Cc:
Release Notes:
API Changes:

Description

Implementation of customized workflow (see NewWorkflow) requires that a component has two owners: developer-owner and QA-owner.

This feature requires database changes. If the database upgrade is performed in a patch, it is very hard to track changes and upgrade projects with new version of Trac.

On the other hand, this feature does not make any impact on existing system.

Thus, it is necessary to implement the change in mainstream for simplifying further development on customized workflow.

Necessary changes:

  • trac/db_default.py: add column qaowner to table component. Modify default database data;
  • trac/upgrades/dbX.py: An upgrade script that sets qaowner equals to currnet owner during the upgrade;
  • trac/admin/scripts.py: Extend commands for component modification:
    • component add <name> <owner> [<qaowner>]: Set qaowner if specified. Set qaowner equals to owner if omitted.
    • component chown <name> <owner> [<qaowner>]: Change qaowner if specified. Set qaowner equals to owner if omitted.

(In preparation for #869)

Attachments

qaowner-r1674.diff (5.6 KB) - added by pkou at ua.fm 7 years ago.
Patch for the changes
Load_A4.ctl (2.1 KB) - added by anonymous 6 years ago.

Download all attachments as: .zip

Change History

comment:1 Changed 7 years ago by pkou at ua.fm

Would you agree with this change for 0.9?

Changed 7 years ago by pkou at ua.fm

Patch for the changes

comment:2 Changed 7 years ago by pkou at ua.fm

Patch attached.

While working on the patch: Milestone owner has been removed in [1169] because of #786. Why? This creates another minor problem for customized workflows. Need to think how it will look without milestone owner.

comment:3 Changed 7 years ago by cmlenz

What do you mean with "milestone owner has been removed"? We never had owners for milestones AFAIK.

comment:4 Changed 7 years ago by pkou at ua.fm

Oops! Just entangled with all my patches. Sorry :) (Do we have a possibility to edit/remove comments? :)
Milestone owner has been introduced with my first patch for NewWorkflow.

comment:5 Changed 7 years ago by mgood

  • Severity changed from normal to enhancement

comment:6 Changed 7 years ago by cmlenz

  • Resolution set to wontfix
  • Status changed from new to closed

Okay, I'd rather not check in this change separately as it'd just add unused code/db column. Let's just keep it together with the NewWorkflow patch (#869), okay?

Changed 6 years ago by anonymous

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
to The owner will be changed from cmlenz. Next status will be 'closed'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.