Skip to content

Fix security logs av#661

Open
AntonV1211 wants to merge 22 commits into
devfrom
fix_security_logs_av
Open

Fix security logs av#661
AntonV1211 wants to merge 22 commits into
devfrom
fix_security_logs_av

Conversation

@AntonV1211
Copy link
Copy Markdown
Contributor

Comment thread js/src/spbc-table.js Outdated
Comment thread lib/CleantalkSP/SpbctWP/ListTable.php
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the Security Log “User activity timeline” to support a “Most Active Users” view and adds a user-based filter that propagates from the timeline selection into the Security Logs list table (pagination/sort).

Changes:

  • Added server-side handling to apply a users filter to list-table SQL args for AJAX pagination/sort/switch.
  • Enhanced timeline widget UX: “Most Active Users” option, richer user labels (role + event count), and click-to-filter behavior that updates URL query params.
  • Updated table pagination JS to include the users filter in AJAX requests and added an explicit updateSecurityLogTable() helper.

Reviewed changes

Copilot reviewed 5 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
lib/CleantalkSP/SpbctWP/ListTable.php Adds applyUserFilter() and applies it to AJAX table handlers.
js/src/spbc-timeline-widget.js Adds “Most Active Users” view and URL-param-driven filtering behavior.
js/src/spbc-table.js Refactors pagination listeners and propagates users filter into pagination requests; adds updateSecurityLogTable().
inc/spbc-tools.php Extends timeline data payload to include user roles and adjusts timestamp offset handling.
inc/spbc-admin.php Removes registration for wp_ajax_spbc_tbl-switch.
js/spbc-timeline-widget.min.js Built/minified output update for timeline widget.
js/spbc-timeline-widget.min.js.map Source map update for timeline widget.
js/spbc-table.min.js Built/minified output update for table script.
js/spbc-table.min.js.map Source map update for table script.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lib/CleantalkSP/SpbctWP/ListTable.php
Comment thread lib/CleantalkSP/SpbctWP/ListTable.php Outdated
Comment thread js/src/spbc-table.js
Comment thread inc/spbc-admin.php
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.

4 participants