-
-
Notifications
You must be signed in to change notification settings - Fork 173
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 Flow integration #1138
Conversation
40d638e
to
0ea52a8
Compare
ed993f4
to
474da6b
Compare
* @inheritDoc | ||
*/ | ||
public function isAvailableForScope(int $scope): bool { | ||
return true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this also allows administrator to configure global rules for creating bookmarks. Is this intended?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I wasn't sure what the use case for this would be, but I thought it couldn't hurt.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the parameter is one of the scope constants from IManager, SCOPE_ADMIN
or SCOPE_USER
. Some operations might be available for both scopes, others like running arbitrary scripts only for admins, because they could do anything, and others like posting to Talk is personal only as it is bound to the user who configured the rule.
@blizzz I'd like to add some events here, too, but I'm kinda at a loss on how I would go about that. I already expose events on the global dispatcher. Do I merge those with the flow events? Or are those different events? What would be the event subject? Can you guide me through this a little?