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

Track log entries instead of hooks for log actions #49

Open
Dinoguy1000 opened this issue Jun 4, 2023 · 1 comment
Open

Track log entries instead of hooks for log actions #49

Dinoguy1000 opened this issue Jun 4, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@Dinoguy1000
Copy link
Contributor

The current hook-based design of this extension means it can only show actions that trigger a hook, and data accessible via that hook (see #23, #28, #30, probably other things no one's noticed or bothered to file an issue for). I'm assuming this won't be a straightforward drop-in replacement, so definitely more of a wishlist item than anything.

@jayktaylor jayktaylor added the enhancement New feature or request label Jun 6, 2023
@jayktaylor
Copy link
Owner

You're right - this won't be a straight-forward change.

Moving to a system where we instead pipe the RC feed output into a Discord channel is not a terrible idea - in fact, there is an alternative extension that already does this - but it doesn't feel like it's currently worth the investment in doing so for this extension. I understand that there will be some functionality that is missing because we're working directly with MediaWiki's hooks, which is a shame. Recently, hooks integrated into MW core have started getting better at providing more information to third-party extensions, so hopefully more things become possible using hooks only.

I think what I like most about the approach that we take in this extension is that it doesn't really require much configuration at all, which makes it perfect for shared hosts, farms, and people who aren't very knowledgable in server administration. It basically just requires a vanilla PHP setup. In contrast, using $wgRCFeeds requires the sockets PHP extension to be enabled.

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

No branches or pull requests

2 participants