Modify

Opened 7 years ago

Closed 4 years ago

Last modified 4 years ago

#8140 closed task (fixed)

Merge codebase with TracTicketStatsPlugin?

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Component: TicketStatsMacro
Severity: normal Keywords:
Cc: Steffen Hoffmann Trac Release: 0.11

Description

I'm working on fixing open issues with the TracTicketStatsPlugin, and since these two plugins share so much code, I was wondering how you'd feel about packaging them together.

Attachments (0)

Change History (10)

comment:1 Changed 7 years ago by anonymous

Yes, will be wonderfull. Most of the code was taken from that plugin. I should have done when I created the plugin but I didn't felt very comfortable with the API trac.

Please tell me if you need some help.

comment:2 Changed 6 years ago by Ryan J Ollos

If the following is okay with you, I propose to:

Later on, I intend to merge the TracTicketStatsPlugin with the TracMetrixPlugin, and rename it ProjectMetricsPlugin. I also hope to add functionality and merge additional plugin functionality, pending buy-in by other plugin authors.

comment:3 Changed 6 years ago by Ryan J Ollos

I've been committing to TracTicketStatsPlugin for some time, but did not fully go through the correct channels to adopt the plugin, so I've initiated that process in #7537.

comment:4 Changed 4 years ago by Ryan J Ollos

Owner: changed from Guillermo M Narvaja to Ryan J Ollos
Status: newassigned

comment:5 Changed 4 years ago by Ryan J Ollos

(In [13096]) Refs #8140: Fixed PEP-0008 violations.

comment:6 Changed 4 years ago by Ryan J Ollos

(In [13099]) Refs #8140: Merged the TicketStatsMacro into the TracTicketStatsPlugin. The TicketStatsMacro is now deprecated and development will continue in the TracTicketStatsPlugin codeline.

Bumped the version of the TracTicketStatsPlugin to 3.0.0dev and set the default Yahoo YUI API version to be 2.9.0, which is the version that the TicketStatsMacro used.

comment:7 Changed 4 years ago by Ryan J Ollos

Resolution: fixed
Status: assignedclosed

comment:8 Changed 4 years ago by Ryan J Ollos

(In [13108])

Refs #8140:

  • Changed macro option name from ticketstatsmacro to ticketstats.
  • Moved macro template to an external file. When only Trac 0.12 and later is supported, and add_script_data is available, the template can be transformed to a JS script.

comment:9 Changed 4 years ago by Ryan J Ollos

(In [13110]) Refs #8140: Tag version 2.2.0 of TracTicketStatsPlugin.

comment:10 Changed 4 years ago by Ryan J Ollos

(In [13197]) Refs #8140: Added requirement of TracAdvParseArgsPlugin since the code integrated from the TicketStatsMacro requires it. However, this dependency might be eliminated later on.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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

 
Note: See TracTickets for help on using tickets.