Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement a Core Alert plugin with methods to use it from non-core plugins #72

Open
MKhayle opened this issue May 19, 2022 · 3 comments
Labels
core related to core functionality enhancement New feature or request Priority P3 ordinary flow of work

Comments

@MKhayle
Copy link

MKhayle commented May 19, 2022

Describe the solution you'd like
A Core feature to add an alert plugin system for automated, event & time-based triggers with methods to support external alerts from potential non-core plugins such as an Event plugin (#78) or a Weather plugin (#87).

Describe alternatives you've considered
Implement it as a non-core plugin for alerts if considered unfit as a Core feature.

Additional context
context

@MKhayle MKhayle added the enhancement New feature or request label May 19, 2022
@project-automations project-automations bot moved this to Todo ✏️ in MVP May 19, 2022
@karashiiro
Copy link
Contributor

This is a good idea, maybe we could wrap a proper scheduler along the lines of Quartz.NET, and let plogons modules register jobs with that wrapper.

@MKhayle MKhayle changed the title Implement an Alert module as a Core module with methods to use it from non-core modules Implement a Core Alert module with methods to use it from non-core modules May 19, 2022
@karashiiro
Copy link
Contributor

The only issue is that almost everything seems to use Redis exclusively as a backing store.

@karashiiro
Copy link
Contributor

I'm assuming that comparison table is a comprehensive list of well-maintained solutions (it may not be), which case we'd have a few options with different costs:

  1. Use something off-the-shelf. This is the easiest option, but it means adding another service to our deployment model.
  2. Use something off-the-shelf and find/create a translation layer that allows us to use postgres (this doesn't seem to exist, although the opposite does. Designing this would probably be more complex than it sounds.
  3. Create our own scheduler that can use postgres.

@karashiiro karashiiro added the core related to core functionality label May 19, 2022
@MKhayle MKhayle changed the title Implement a Core Alert module with methods to use it from non-core modules Implement a Core Alert plugin with methods to use it from non-core plugins May 20, 2022
@kalilistic kalilistic added the Priority P3 ordinary flow of work label May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core related to core functionality enhancement New feature or request Priority P3 ordinary flow of work
Projects
Status: Todo ✏️
Development

No branches or pull requests

3 participants