Plugin dev help needed for possible dedicated CP plugin

I have made a fork of a small email logging plugin for my own use. I pruned out all the upsells to the pro version and simplified it in places, and it’s working nicely on all my sites. Code is here: https://github.com/simplycomputing/sc-email-log

I would be happy to make this a dedicated CP plugin and let others use it, but I am not a developer and I’m not confident in releasing this out to a wider audience. In particular, some of the reservations I have are:

  1. I have cleaned out some code but I’m sure there’s other stuff that could be removed and tidied up.
  2. I don’t know about security issues so can’t tell if the original code was secure, or even well-written.
  3. If someone reported an issue I would have no idea where to start looking to fix it.
  4. It works at the moment with CP but I wouldn’t be able to fix any future compatibility issues.

So, I’m just wondering if there is some developer who would volunteer to help with the technical side of this. I can push updates (it is managed by codepotent’s Update Manager) and do any necessary leg work, but I don’t have the coding skills to feel able to put this out. So I would need a dev to look it over and possibly clean it up, then be available for the occasional questions down the track.

Or, if there is a plugin developer who wants to take it on as their own project then by all means go for it. :kissing:

The plugin logs all the emails sent by ClassicPress. It has a dashboard widget showing the number of emails saved, and a feature to set a limit and notify admin when it’s time for a clean-up. Some screenshots:

8 Likes

I’ll take a look at it over the weekend.

4 Likes

Interesting, this is very useful and I even made a petition for one: https://petitions.classicpress.net/posts/190/add-email-logging-feature

We can put it under https://github.com/ClassicPress-research if there are no individual takers. Practically this still wouldn’t change much until other people start working on it, but it would provide a way forward for collaboration to happen more easily and to make this a more officially supported plugin once that happens.

3 Likes

Wouldnt take much, perhaps make it user composer autoloader instead of the self hacked autoloader thing its got going on there.

1 Like

Thanks @Pross - it was a fork of this one: https://wordpress.org/plugins/email-log/

I thought the petition was a very good idea @james - the one I was using by Christian Zoller was taken over by MailPoet and it’s now >WP5.0. So I needed an alternative.

2 Likes

Heres what i have so far, isnt finished yet
Whats Ive done so far:

  • Got rid of all the loadie stuff and implemented composer autoload.
  • Switched to core jquery-ui
  • Started the phpcs cleanup with wpcore rulesets, WIP here probably finish this over the weekend.
3 Likes

Great, thanks @Pross. As I mentioned, I’m not a dev so I don’t understand any of that. :zipper_mouth_face:

What do you think about James’ suggestion to move it to ClassPress-reseach. Makes sense to me if it’s going to be a community effort.

1 Like

Yea seems like it would be a good idea

1 Like

OK. Not sure of the best way to do this, but if I ping @james I’m sure he’ll have it done in nanoseconds.

Edit: I’d suggest calling it something like cp-email-log.

1 Like

Yea maybe all cp adopted plugins should start with the same prefix

2 Likes

Ok, I copied it here, and added you both as collaborators: https://github.com/ClassicPress-research/cp-email-log

I took @ozfiddler’s repo and then stuck the latest changes from @Pross on top of it as a separate commit, to avoid starting the commit history over again. This will make it easier to review what changed going forward.

Note this isn’t an officially supported plugin yet, we’d need someone to lead it and you’ll have to talk to our new Ecosystem team lead also!

3 Likes

Right, pinging @1stepforward, our ecosystem man. Once he’s sorted out that minor Climate Change issue he might be able to have a look at this.

3 Likes