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

Feature/events dispatching available on Monitor model #150

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

alessandrofuda
Copy link

Hi,
for my need I had to intercept some user action from the UI (relative to queue_monitor table, Monitor model).
For example, when a user delete an item from UI, I have to implement some action in my application.

So I propose to manage Events fired by the Monitor model (on deleting and creating records).

So, developers can makes their Listeners, on application side, "hooking" to package's Events.

In short: make available 4 main Events we can use in our listeners in our application.

Let me know if might be useful, thanks

(in the pr , restore .gitignore and composer.json)

@romanzipp
Copy link
Owner

Hey! Thanks for your contribution. The PR itself looks good but I'm currently debating if a custom Monitor class which can be overridden by the consuming application would be a good idea.
Nonetheless, it looks like you have deleted the composer.json, I assume that was a mistake?

@alessandrofuda
Copy link
Author

it looks like you have deleted the composer.json, I assume that was a mistake?

Yes, sure is my fault :)

@alessandrofuda
Copy link
Author

it looks like you have deleted the composer.json, I assume that was a mistake?

Yes, sure is my fault :)

Restored composer.json and .gitignore

@alessandrofuda
Copy link
Author

I'm currently debating if a custom Monitor class which can be overridden by the consuming application would be a good idea.

Yes, for the moment I use my custom Monitor class via model value into config/queue-monitor.php. My custom model extends package Monitor model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants