Modify

Opened 5 years ago

Closed 2 years ago

Last modified 8 months ago

#5715 closed defect (wontfix)

No documentation on WikiMacros page

Reported by: rjollos Owned by: rjollos
Priority: high Component: WikiStatsPlugin
Severity: normal Keywords: macro description
Cc: Trac Release: 0.11

Description (last modified by rjollos)

There is no documentation for this plugin on the WikiMacros page for an instance of Trac where the macro is installed. It would be nice to add a short entry describing how to use each macro. See attached screen capture.

Screen capture showing no documentation on WikiMacros page

Attachments (1)

UserStatsMacroNoDocs.png (30.4 KB) - added by rjollos 5 years ago.
Screen capture showing no documentation on WikiMacros page

Download all attachments as: .zip

Change History (13)

Changed 5 years ago by rjollos

Screen capture showing no documentation on WikiMacros page

comment:1 Changed 5 years ago by rjollos

  • Component changed from UserStatsMacro to WikiStatsPlugin

comment:2 Changed 5 years ago by anonymous

  • Owner changed from pedro to Rottenchester

comment:3 Changed 4 years ago by rjollos

  • Owner changed from Rottenchester to rjollos
  • Priority changed from normal to high
  • Status changed from new to assigned

comment:4 Changed 3 years ago by rjollos

  • Status changed from assigned to new

comment:5 follow-up: Changed 3 years ago by hasienda

  • Keywords macro description added

For various reasons I'm aggressively rebuilding WikiStatsPlugin functionality inside AccountManagerPlugin (see #9852).

For what it's worth, please take a look at it's embedded macro descriptions. Comments welcome.

comment:6 in reply to: ↑ 5 ; follow-up: Changed 3 years ago by rjollos

Replying to hasienda:

For various reasons I'm aggressively rebuilding WikiStatsPlugin functionality inside AccountManagerPlugin (see #9852).

Thanks for doing that, I think it will be a very nice addition to the AccountManagerPlugin. I'm going to follow though on the adoption for this plugin, fix the remaining open issues and mark it as soon to be deprecated, in order to direct attention to the AccountManagerPlugin.

For what it's worth, please take a look at it's embedded macro descriptions. Comments welcome.

It is looking very good. I've never liked that the WikiStatsPlugin has a macro named Stats, feeling that it is too generic of a name. Perhaps ProjectStats, or TracStats? Off hand, I'd prefer ProjectStats. This would also prevent a collision in the case that both macros are installed. Or are you aiming for backward compatibility with this plugin?

comment:7 in reply to: ↑ 6 Changed 3 years ago by hasienda

Replying to rjollos:

Replying to hasienda:

For various reasons I'm aggressively rebuilding WikiStatsPlugin functionality inside AccountManagerPlugin (see #9852).

Thanks for doing that, I think it will be a very nice addition to the AccountManagerPlugin. I'm going to follow though on the adoption for this plugin, fix the remaining open issues and mark it as soon to be deprecated, in order to direct attention to the AccountManagerPlugin.

Great you like it. But for the other part I hoped to discourage you enough at least to not take #3874. After reading the code I think it's worth to keep the idea. The implementation is a bit too straight-forward, i.e. can't make use of new caching capability in Trac 0.12, and more. I'll try my best to warrant depreciation of this fine plugin in favor of the new AcctMgr wiki macro suite.

... Comments welcome.

It is looking very good. I've never liked that the WikiStatsPlugin has a macro named Stats, feeling that it is too generic of a name. Perhaps ProjectStats, or TracStats? Off hand, I'd prefer ProjectStats. This would also prevent a collision in the case that both macros are installed. Or are you aiming for backward compatibility with this plugin?

Yes, I'd like to take the hurdle of supplying backwards-compatibility for someone with hundreds of existing macro invocations. OTOH, replacing just the name and keeping a compatible parameter set could be an acceptable compromise. I confess, that I liked the old short name for shortness, but it's truly unspecific wording and absence of CamelCase is unsatisfying. I trust you for the choice in general: Stats will become ProjectStats for AcctMgr offerings soon.

Btw, I did look at TracStatsPlugin the first time after you mentioned it here, great stuff. But external site and requirement of a GitHub account has nearly scared me away from creating an issue. Hope we'll gain more on the t-h.o side to convince people to a least to mirror their repositories here again.

comment:8 Changed 2 years ago by rjollos

Should we deprecate this plugin, close all of the open tickets and direct users to the AccountManagerPlugin? I think that comment:7 is suggesting this approach.

My thinking is:

  • If a user wants this macro but not the account manager features, they can install AccountManagerPlugin and only enable the macros.
  • One of the macros depends on AccountManagerPlugin anyway.
  • I have too much to maintain and I'm trying to dump things that have no value. Let's drive users towards the AccountManagerPlugin.

Thoughts?

comment:9 Changed 2 years ago by hasienda

I'll try to make WikiMacros inside of AccountManagerPlugin as independent as possible from the other modules and will seek graceful degradation, if possible.

More elaborated statistics certainly involve heavy data processing (see my related comments to #3874). So for me it feels better to move to AccountManager or even TracStatsPlugin for that purpose, than to maintain a somewhat half-hearty and not portable hack.

IMHO depreciation with pointing to alternative and in parts already superior solution would be a reasonable decision.

comment:10 Changed 2 years ago by rjollos

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

This plugin is deprecated, and a notice has been posted to the WikiStatsPlugin page with more details. Please comment here if you have any concerns.

comment:11 Changed 2 years ago by hasienda

Fine for me.

comment:12 Changed 8 months ago by rjollos

  • Description modified (diff)

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.