From 69659f8824385190c6bb6ee4d98e4eca04749adc Mon Sep 17 00:00:00 2001 From: smarcet Date: Fri, 25 Sep 2026 03:19:09 -0300 Subject: [PATCH] fix(audit-log): filter event audit logs by every SummitEvent subclass summit-api emits audit.entity as the concrete subclass name, so the class_name==SummitEvent filter only matched plain events and hid every Presentation, SummitEventWithFile and SummitGroupEvent record. The post-save refresh in event-actions still sent the legacy SummitEventAuditLog class name, which matches nothing on the audit-logs-api and blanked the panel after saving an event. The global audit log page, the event form audit panel and the post-save refresh now send SummitEvent||Presentation||SummitEventWithFile||SummitGroupEvent. --- src/actions/event-actions.js | 5 ++++- src/components/forms/event-form/index.js | 5 ++++- src/pages/audit-log/__tests__/audit-log-page.test.js | 6 ++++-- src/pages/audit-log/audit-log-page.js | 4 +++- 4 files changed, 15 insertions(+), 5 deletions(-) diff --git a/src/actions/event-actions.js b/src/actions/event-actions.js index e0ea6c8c8..2d94ea1f7 100644 --- a/src/actions/event-actions.js +++ b/src/actions/event-actions.js @@ -587,7 +587,10 @@ export const saveEvent = (entity, publish) => async (dispatch, getState) => { } dispatch( getAuditLog( - [`event_id==${entity.id}`, "class_name==SummitEventAuditLog"], + [ + `event_id==${entity.id}`, + "class_name==SummitEvent||Presentation||SummitEventWithFile||SummitGroupEvent" + ], null, DEFAULT_CURRENT_PAGE, DEFAULT_PER_PAGE diff --git a/src/components/forms/event-form/index.js b/src/components/forms/event-form/index.js index 25f761796..7dd44b7bb 100644 --- a/src/components/forms/event-form/index.js +++ b/src/components/forms/event-form/index.js @@ -2241,7 +2241,10 @@ class EventForm extends React.Component { > {entity.id !== 0 && ( diff --git a/src/pages/audit-log/__tests__/audit-log-page.test.js b/src/pages/audit-log/__tests__/audit-log-page.test.js index 05ad55d6a..b0220272d 100644 --- a/src/pages/audit-log/__tests__/audit-log-page.test.js +++ b/src/pages/audit-log/__tests__/audit-log-page.test.js @@ -43,11 +43,13 @@ describe("AuditLogPage", () => { ); }); - it("scopes AuditLogs to the standalone context, filtered to SummitEvent audit logs", () => { + it("scopes AuditLogs to the standalone context, filtered to every SummitEvent subclass", () => { renderPage(0); const props = JSON.parse(screen.getByTestId("audit-logs-mock").textContent); expect(props.filterId).toBe("standalone"); - expect(props.entityFilter).toEqual(["class_name==SummitEvent"]); + expect(props.entityFilter).toEqual([ + "class_name==SummitEvent||Presentation||SummitEventWithFile||SummitGroupEvent" + ]); }); }); diff --git a/src/pages/audit-log/audit-log-page.js b/src/pages/audit-log/audit-log-page.js index 425475f32..908171cf3 100644 --- a/src/pages/audit-log/audit-log-page.js +++ b/src/pages/audit-log/audit-log-page.js @@ -23,7 +23,9 @@ const AuditLogPage = ({ totalLogEntries }) => ( );