Skip to content

Conversation

@samarp-jain
Copy link
Contributor

No description provided.

samarp-jain and others added 4 commits June 16, 2025 21:35
- Introduced new AuditLogs component for displaying audit data.
- Implemented getAuditData service to fetch audit logs based on various parameters.
- Added AuditFilterModal for filtering options in the audit logs.
- Updated migration routes to include endpoint for fetching audit data.
- Enhanced settings component to integrate audit logs functionality.
- Added necessary styles and interfaces for the new components.
…nd audit log components

- Updated migration routes and service to use optional chaining for safer access to properties.
- Enhanced AuditLogs component with optional chaining to prevent potential runtime errors.
- Ensured consistent handling of undefined values across various data access points.

This comment was marked as outdated.

@umeshmore45 umeshmore45 requested a review from Copilot June 17, 2025 09:22
Copy link
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 adds support for audit logs in both the UI and API layers, along with new Docker configuration updates for the migration services. Key changes include the introduction of the AuditLogs component and related styling in the UI, new API endpoints and service logic to fetch audit data, and updates to Docker and environment configuration files.

Reviewed Changes

Copilot reviewed 26 out of 26 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
ui/src/components/Common/Settings/index.tsx Added AuditLogs component import and tab logic; minor code cleanups.
ui/src/components/Common/Settings/Settings.scss Updated styles for borders and layout adjustments.
ui/src/components/AuditLogs/* New AuditLogs related components, styles, and interfaces.
ui/src/components/AuditFilterModal/* Added AuditFilterModal component and styles for filtering audit data.
ui/src/cmsData/setting.json Added audit_logs configuration for CMS settings.
Dockerfile, setup-docker.sh, docker-compose.yml, .dockerignore New and updated Docker and compose configurations for the apps.
api/src/services/migration.service.ts Introduced getAuditData service and updated data transformation logic.
api/src/routes/migration.routes.ts, api/src/controllers/migration.controller.ts Added new API routes and controller for audit data.
api/src/constants/index.ts Added constants for GET_AUDIT_DATA used by new audit log API.
api/Dockerfile, api/.dockerignore Docker configurations and ignore files updated for the API.
.talismanrc, .dockerignore Updated ignore and checksum configurations.
Comments suppressed due to low confidence (1)

api/src/services/migration.service.ts:685

  • The string 'Entries_Select_feild' appears to have a typo; please verify if it should be 'Entries_Select_field'.
moduleName === 'Entries_Select_feild' ? transformedData = transformedData?.filter((log) => {

@sayalijoshi27 sayalijoshi27 merged commit 291c4a9 into pre-dev Jun 17, 2025
7 checks passed
@sayalijoshi27 sayalijoshi27 deleted the feature/audit-logs branch June 17, 2025 09:58
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.

6 participants