Implement Alerts as an Entity and remove the node integration.

CommentFileSizeAuthor
#5 user_alert.tar_.gz18.85 KBbernman

Comments

kevinquillen’s picture

kevinquillen commented
Assigned: Unassigned » kevinquillen
kevinquillen’s picture

kevinquillen commented
Status: Active » Postponed
bernman’s picture

bernman commented

I've taken an initial stab at converting the module to be fully entity based. As a consequence, since the scheduler module only works with nodes, the module now needs to provide its own Scheduling functionality (which I borrowed liberally from the Scheduler module). The new version does not provide any upgrade path from the existing module.

If you are interested, I can provide additional details as desired.

kevinquillen’s picture

kevinquillen commented

It's been on my list a while but have been pretty busy with project work. On the upside, I have a much better grasp on the Entity system. Would appreciate any help.

bernman’s picture

bernman commented
StatusFileSize
new user_alert.tar_.gz18.85 KB

I'm attaching a tarfile of the code as it's such a major rewrite that a patch file seems inappropriate. Hope that's ok. The code could use some additional cleanup and some admin settings that would be useful are not in there yet. Still, it seems functional. Happy to help.

kevinquillen’s picture

kevinquillen commented
Status: Postponed » Needs review

I'll be able to take a look at this this week.

kevinquillen’s picture

kevinquillen commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)

See 2.x branch.