Skip to content

ATLAS-5350: Atlas UI: Enhance Purge Audit Results UI - #708

Merged
pawarprasad123 merged 9 commits into
apache:masterfrom
Brijesh619:ATLAS-5350
Aug 11, 2026
Merged

ATLAS-5350: Atlas UI: Enhance Purge Audit Results UI#708
pawarprasad123 merged 9 commits into
apache:masterfrom
Brijesh619:ATLAS-5350

Conversation

@Brijesh619

@Brijesh619 Brijesh619 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?
This patch introduces functional UI enhancements to the Audit view for both the Classic (dashboardv2) and React dashboards, specifically targeting Purge operation summaries and TYPE_DEF rendering improvements.

Summary of Changes:

  • SUMMARY & runId Filtering: Implemented seamless integration for runId based filtering. When a user filters by runId, the UI automatically strips conflicting query parameters and safely injects the SUMMARY criteria to fetch the correct overarching audit record.
  • Purge Metric Cards: Added a 4-card dashboard (REQUESTED, PURGED, FAILED, SKIPPED) to clearly summarize Purge operations. The cards accurately aggregate base entities and dependencies (purgedCount + purgedDependenciesCount) and provide contextual alerts (e.g., highlighting failures and directing users to purgefailure.log).
  • Client-side Drawer Pagination: Added an interactive side drawer that slides out when interacting with the Purge Summary cards. This drawer utilizes highly efficient client-side drawer pagination and virtualization to gracefully render massive arrays of GUIDs without endlessly hanging the browser.
  • auditRowKind UI Hiding: Cleaned up the frontend UI to properly hide internal implementation details by masking auditRowKind elements from the end-user.
  • Robust Copy-to-Clipboard: Enhanced the "Copy Run ID" action to work reliably across all browser environments. Both the main UI and the side drawer now feature independent, immediate "Copied!" visual feedback.
  • Accessibility (a11y) Upgrades: The Purge metric cards are now fully navigable by keyboard and screen readers, supporting standard Enter/Space interactions.
  • TYPE_DEF Audit Rendering Changes: Refined the rendering logic for TYPE_DEF audits to ensure a more robust and visually consistent audit trail representation across both dashboards.

How was this patch tested?

  • Automated Tests: Augmented the Jest suite with extensive tests covering the new summary dashboard, empty drawer states, graceful degradation during API timeouts, and edge cases around drawer limit inputs.
  • Manual UI Tests:
    • Triggered a Purge operation and verified the dynamic metric cards correctly aggregate base and dependent entity counts.
    • Verified that the client-side pagination correctly limits DOM nodes and handles high-volume drawer scrolling smoothly.
    • Ensured perfect feature parity between the Atlas React UI and the Classic (dashboardv2) UI.

@Brijesh619

Copy link
Copy Markdown
Contributor Author

Atlas React UI

Screenshot from 2026-07-30 11-06-33 Screenshot from 2026-07-30 11-06-00 Screenshot from 2026-07-30 11-05-37 Screenshot from 2026-07-30 11-05-30 Screenshot from 2026-07-30 11-04-07

Atlas Classic UI

Screenshot from 2026-07-30 11-12-07 Screenshot from 2026-07-30 11-11-21 Screenshot from 2026-07-30 11-11-14 Screenshot from 2026-07-30 11-11-03

@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-07-30 11-47-34

@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-07-30 12-06-12

@Brijesh619
Brijesh619 force-pushed the ATLAS-5350 branch 2 times, most recently from 44904d0 to be0d3a8 Compare July 30, 2026 10:06
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboardv2/public/css/scss/style.scss Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboardv2/public/js/templates/audit/DrawerView_tmpl.html
@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-08-07 16-11-38 Screenshot from 2026-08-07 16-11-17 Screenshot from 2026-08-07 16-10-58

@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-08-07 16-36-08

@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-08-07 17-03-14

@pawarprasad123 pawarprasad123 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Update Pr description properly as per the final changes

Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/__tests__/AuditResults.test.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/__tests__/AuditResults.test.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx
Comment thread dashboard/src/hooks/__tests__/useVirtualization.test.ts
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx

@pawarprasad123 pawarprasad123 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

after fixing the changes, also check for test case.

Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboardv2/public/js/views/audit/AdminAuditTableLayoutView.js
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx
Comment thread dashboard/src/hooks/__tests__/useVirtualization.test.ts
@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-08-10 11-52-33 Screenshot from 2026-08-10 11-51-57 Screenshot from 2026-08-10 11-51-45 Screenshot from 2026-08-10 11-51-39 Screenshot from 2026-08-10 11-51-32 Screenshot from 2026-08-10 11-50-58 Screenshot from 2026-08-10 11-50-38 Screenshot from 2026-08-10 11-50-32 Screenshot from 2026-08-10 11-49-27 Screenshot from 2026-08-10 11-02-42

Comment thread dashboardv2/public/js/views/audit/AdminAuditTableLayoutView.js Outdated
Comment thread dashboardv2/public/js/views/audit/AdminAuditTableLayoutView.js
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx Outdated
Comment thread dashboardv2/public/css/scss/drawer.scss
Comment thread dashboard/src/hooks/__tests__/useVirtualization.test.ts
Comment thread dashboard/src/views/Administrator/Audits/AuditResults.tsx
@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-08-10 19-10-41

@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-08-10 19-24-20

@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-08-11 01-00-22

@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-08-11 01-26-25

@Brijesh619

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-08-11 10-33-58

@pawarprasad123 pawarprasad123 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@pawarprasad123
pawarprasad123 merged commit fba3527 into apache:master Aug 11, 2026
2 checks passed
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.

2 participants