Modify

Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#60 closed enhancement (fixed)

Make DiscussionPlugin use web admin plugin

Reported by: athomas Owned by: Blackhex
Priority: high Component: DiscussionPlugin
Severity: normal Keywords:
Cc: blackhex@… 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 Blackhex 9 years ago.
tracdiscussion to discussion directory patch
setup.py.diff (1.4 KB) - added by Blackhex 9 years ago.
setup.py patch
discussionplugin.diff (14.1 KB) - added by Blackhex 9 years ago.
Diff against SVN version, substitude previous two diffs.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 9 years ago by athomas

  • Priority changed from normal to high

comment:2 Changed 9 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 9 years ago by athomas

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 9 years ago by athomas

  • milestone 0.9 deleted

Milestone 0.9 deleted

comment:5 Changed 9 years ago by anonymous

  • Cc blackhex@… added

comment:6 Changed 9 years ago by Blackhex

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 9 years ago by athomas

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 9 years ago by Blackhex

tracdiscussion to discussion directory patch

Changed 9 years ago by Blackhex

setup.py patch

comment:8 Changed 9 years ago by Blackhex

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 9 years ago by Blackhex

Diff against SVN version, substitude previous two diffs.

comment:9 Changed 8 years ago by Blackhex

  • Owner changed from athomas to Blackhex
  • Status changed from new to assigned
  • Trac Release set to 0.8

Should be implemented in next changeset.

comment:10 Changed 8 years ago by Blackhex

  • Resolution set to fixed
  • Status changed from assigned to closed

Implemented in chageset 836.

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.