Opened 12 years ago
Closed 8 years ago
#10944 closed task (worksforme)
Review user ID changer component
Reported by: | Steffen Hoffmann | Owned by: | Radek Bartoň |
---|---|---|---|
Priority: | normal | Component: | ScreenshotsPlugin |
Severity: | minor | Keywords: | needinfo rename user |
Cc: | Ryan J Ollos | Trac Release: | 0.11 |
Description
Please decide, if you would like to take the upcoming user ID change component into your plugin, or leave it as part of the AccountManagerPlugin code distribution.
I open this ticket in order to notify on the existence of this component for ScreenshotsPlugin and get explicit feedback from plugin's author. Thank you in advance for taking your time to review that code, maybe even the underlying concept.
Attachments (0)
Change History (3)
comment:1 follow-up: 2 Changed 12 years ago by
comment:2 Changed 12 years ago by
Replying to hasienda:
With the publication of this code the requests for feedback are almost obsoleted, ...
Note that this doesn't apply here, only for the other hacks, that provided user ID change support up to some degree before.
comment:3 Changed 8 years ago by
Resolution: | → worksforme |
---|---|
Status: | new → closed |
Moving the code to ScreenshotsPlugin sounds like a good idea. However, as a next step I think we should aim to integrate the user management panel and AccountManagerPlugin APIs to Trac, then have ScreenshotsPlugin implement the user ID changer against the Trac API.
(In [12736]) AccountManagerPlugin: Add optional user ID changer components for other plugins, refs #10900, #10901, #10902, #10944 and #10945.
In addition to VotePlugin (supported by RenameTracUsersScript too) support for
is provided within this plugin.
With the publication of this code the requests for feedback are almost obsoleted, even more, because non of the pre-existing solutions provided the coverage and extensibility, that is offered now within this plugin.