feat(audit-log): add UI for Audit Log API by knutties · Pull Request #594 · juspay/superposition

knutties · 2025-07-15T16:24:25Z

Problem

The Audit Log feature that lists all changes on the core tables of Superposition is not available for view in the UI. Teams using Superposition need it for easy review - particularly when they want to port the changes to other instances.

Solution

This PR adds a UI over the Audit Log API. It uses the same patterns for tables/filters as the existing pages (particularly the experiment list page).

Environment variable changes

N/A

Pre-deployment activity

N/A

Post-deployment activity

Things needed to be done after deploying this change (if any)

API changes

N/A

Possible Issues in the future

N/A

crates/frontend/src/pages/audit_log/filter.rs

knutties · 2025-09-13T10:49:07Z

crates/experimentation_platform/src/api/experiments/handlers.rs

        default_config::DefaultConfigUpdateRequest,
        experiment_groups::ExpGroupMemberRequest,
        experiments::{
-            ApplicableVariantsQuery, ApplicableVariantsRequest, AuditQueryFilters,


This file's change are removing the audit log that was present in the experimentation schema earlier.

crates/frontend/src/pages/audit_log.rs

Datron · 2025-10-06T07:35:10Z

crates/frontend/src/pages/audit_log.rs

+                        ele_map
+                            .insert(
+                                "timestamp".to_string(),
+                                Value::String(ele.timestamp.format("%v %T").to_string()),


Can this be formatted to IST for now since all our customers are in India

We dont need to do this right now
We need to achieve that throughout the framework

Which can be done later

It can be hardcoded for now, the UTC time is annoying to convert in my head

lets not do it now, I have a solution which is more dynamic and robust based on user's Locale

I am not saying not to do it at all

crates/context_aware_config/src/api/audit_log/handlers.rs

Datron · 2025-10-06T07:39:47Z

crates/frontend/src/pages/audit_log/filter.rs

@@ -0,0 +1,318 @@
+use std::{fmt::Display, str::FromStr};


This should be in the folder crates/frontend/src/components/audit_log_filters.rs

Its fine
Its following as per other already existing ones

Its a component right? Lets put it in the component folder. If others are in the wrong place, we can move those to it another PR. Seems weird otherwise

crates/frontend/src/pages/audit_log.rs

crates/superposition_types/src/api/config.rs

Datron · 2025-10-10T10:02:45Z

clients/haskell/sdk/Io/Superposition/Model/ListAuditLogsInput.hs

superposition/crates/frontend/src/pages/audit_log/mod.rs

Lines 33 to 43 in 5e061aa

Column::new(

"timestamp".to_string(),

false,

move |value: &str, _row: &Map<String, Value>| {

let timestamp = value.to_string();

view! { <span class="text-gray-900">{timestamp}</span> }.into_view()

},

ColumnSortable::No,

Expandable::Disabled,

default_column_formatter,

),

timestamp is still present as the key column. Is this okay?

* feat: add audit log UI with filters * feat: add audit log UI with filters * feat: add audit log UI with filters * feat: add audit log UI with filters * feat: add audit log UI with filters * feat: add audit log UI with filters * feat: audit log ui filter button positioning * feat: move mod.rs out to audit_log.rs * feat: fix compilation errors after rebase * feat: fix UI after rebase * feat: remove audit_log from experiments * feat: separate pagination_params like other pages * feat: use strum for string variants of enum * refactor: Code improvement --------- Co-authored-by: ayush.jain@juspay.in <ayush.jain@juspay.in>

knutties commented Jul 16, 2025

View reviewed changes

crates/frontend/src/pages/audit_log/filter.rs Outdated Show resolved Hide resolved

Datron force-pushed the main branch 4 times, most recently from 36cf78d to be77640 Compare July 21, 2025 13:57

Datron force-pushed the main branch from bc1aecc to 8b62d45 Compare August 4, 2025 11:36

knutties force-pushed the audit-log-ui branch from c5f77fe to 237e867 Compare September 13, 2025 09:03

knutties marked this pull request as ready for review September 13, 2025 09:04

knutties requested a review from a team as a code owner September 13, 2025 09:04

knutties commented Sep 13, 2025

View reviewed changes

knutties force-pushed the audit-log-ui branch 3 times, most recently from 89c6cd2 to 15b4d52 Compare September 17, 2025 16:36

knutties force-pushed the audit-log-ui branch from 0760a0f to 321b4c6 Compare October 3, 2025 16:23

ayushjain17 approved these changes Oct 3, 2025

View reviewed changes

Datron requested changes Oct 6, 2025

View reviewed changes

mahatoankitkumar reviewed Oct 7, 2025

View reviewed changes

crates/frontend/src/pages/audit_log.rs Show resolved Hide resolved

mahatoankitkumar requested changes Oct 7, 2025

View reviewed changes

crates/superposition_types/src/api/config.rs Outdated Show resolved Hide resolved

ayushjain17 force-pushed the audit-log-ui branch from ed16561 to 93dad01 Compare October 7, 2025 08:29

ayushjain17 requested review from Datron and mahatoankitkumar October 7, 2025 08:31

ayushjain17 force-pushed the audit-log-ui branch from 93dad01 to 5facb40 Compare October 7, 2025 13:32

mahatoankitkumar approved these changes Oct 7, 2025

View reviewed changes

Datron reviewed Oct 10, 2025

View reviewed changes

Datron self-requested a review October 14, 2025 10:07

knutties added 5 commits October 14, 2025 15:38

feat: add audit log UI with filters

6cef17e

feat: add audit log UI with filters

da80e99

feat: add audit log UI with filters

d129a7b

feat: add audit log UI with filters

8eb9c89

feat: add audit log UI with filters

8cb11e6

knutties added 8 commits October 14, 2025 15:38

feat: add audit log UI with filters

a9c97c9

feat: audit log ui filter button positioning

6a3cbd7

feat: move mod.rs out to audit_log.rs

dbc4f86

feat: fix compilation errors after rebase

4ba1914

feat: fix UI after rebase

17f54e6

feat: remove audit_log from experiments

d79396b

feat: separate pagination_params like other pages

b6d0974

feat: use strum for string variants of enum

1f81188

Datron approved these changes Oct 14, 2025

View reviewed changes

refactor: Code improvement

73d3cf4

ayushjain17 force-pushed the audit-log-ui branch from 5facb40 to 73d3cf4 Compare October 14, 2025 10:11

ayushjain17 requested a review from Datron October 14, 2025 10:11

Datron approved these changes Oct 14, 2025

View reviewed changes

ayushjain17 added this pull request to the merge queue Oct 14, 2025

Merged via the queue into main with commit 1288eac Oct 14, 2025
12 checks passed

ayushjain17 deleted the audit-log-ui branch October 14, 2025 11:02

	Column::new(
	"timestamp".to_string(),
	false,
	move \|value: &str, _row: &Map<String, Value>\| {
	let timestamp = value.to_string();
	view! { <span class="text-gray-900">{timestamp}</span> }.into_view()
	},
	ColumnSortable::No,
	Expandable::Disabled,
	default_column_formatter,
	),

Conversation

knutties commented Jul 15, 2025

Problem

Solution

Environment variable changes

Pre-deployment activity

Post-deployment activity

API changes

Possible Issues in the future

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ayushjain17 Oct 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

ayushjain17 Oct 6, 2025 •

edited

Loading