Modify

Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#60 closed enhancement (fixed)

Make DiscussionPlugin use web admin plugin

Reported by: Alec Thomas Owned by: Radek Bartoň
Priority: high Component: DiscussionPlugin
Severity: normal Keywords:
Cc: Radek Bartoň Trac Release: 0.8

Description

Remove dependency on AdminConsoleProviderPatch as it doesn't seem likely that it will be merged with Trac, and use the web admin plugin.

Attachments (3)

discussion.diff (10.2 KB) - added by Radek Bartoň 11 years ago.
tracdiscussion to discussion directory patch
setup.py.diff (1.4 KB) - added by Radek Bartoň 11 years ago.
setup.py patch
discussionplugin.diff (14.1 KB) - added by Radek Bartoň 11 years ago.
Diff against SVN version, substitude previous two diffs.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 12 years ago by Alec Thomas

Priority: normalhigh

comment:2 Changed 12 years ago by agave@…

Is this the only outstanding issue with DiscussionPlugin? I'm interested in getting it to work in 0.9 and would like to know if anyone else is working on it and what the issues are. This is the only open ticket I can see.

comment:3 Changed 12 years ago by Alec Thomas

No, it is not the only issue. The biggest issue is that it is only about 10% complete :)

I haven't had time to work on it, so if anybody wants to take over I'd be more than happy to hand over the reins.

comment:4 Changed 11 years ago by Alec Thomas

milestone: 0.9

Milestone 0.9 deleted

comment:5 Changed 11 years ago by anonymous

Cc: Radek Bartoň added; anonymous removed

comment:6 Changed 11 years ago by Radek Bartoň

Blackhex

I made some change in this plugin so I get it worked on 0.9.3 Trac release. But still it is not 100% functional. I'll be working on it further but I dont't know how to contribute my changes. SVN is password protected and sending patch needs someone who merge it to SVN.

comment:7 Changed 11 years ago by Alec Thomas

If you attach your patches to this ticket, I'll take a look and commit them. I've actually been working on it recently, so the code base has changed a fair bit.

Changed 11 years ago by Radek Bartoň

Attachment: discussion.diff added

tracdiscussion to discussion directory patch

Changed 11 years ago by Radek Bartoň

Attachment: setup.py.diff added

setup.py patch

comment:8 Changed 11 years ago by Radek Bartoň

Here they are. I renamed "tracdiscussion" directory to "discussion" because there was some errors with it and removed console.py because of AdminConsoleProviderPatch. There are no new features yet but if I should continue to editing this plugin I'll be working on it. My "working" copy can be view at http://blackhex.no-ip.org/discussion/. Sorry me if patches can't be applyable directly, I don't know any convention with this.

Changed 11 years ago by Radek Bartoň

Attachment: discussionplugin.diff added

Diff against SVN version, substitude previous two diffs.

comment:9 Changed 11 years ago by Radek Bartoň

Owner: changed from Alec Thomas to Radek Bartoň
Status: newassigned
Trac Release: 0.8

Should be implemented in next changeset.

comment:10 Changed 11 years ago by Radek Bartoň

Resolution: fixed
Status: assignedclosed

Implemented in chageset 836.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Radek Bartoň.
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.