feat: add admin activity log viewer - #16
Merged
udaycodespace merged 1 commit intoJul 18, 2026
Merged
Conversation
Contributor
Author
|
@udaycodespace Please do add bonus labels like good-backend , good-pr and good-ui |
udaycodespace
added
ECSoC26
udaycodespace
self-requested a review
July 18, 2026 13:22
Owner
Temporary labels have been assigned based on the PR details. Final labels will be assigned after the review.
|
11 tasks
udaycodespace
added
redo
ecsoc-sentinel
Bot
added
ECSoC26-L2
17 tasks
Contributor
Author
|
@udaycodespace Thanks for the review. Now kindly assign the bonus labels. I have completed the requested validation and added supporting evidence below. Backend validationThe following syntax checks completed successfully: node --check controllers/activityLogController.js
node --check routers/adminRoutes.js
node --check schemas/activityLogModel.jsThe backend also started successfully and connected to MongoDB. Frontend production buildThe frontend production build completed successfully using: npm run build
Authorization verificationVerified the following API behaviour: Search and filtering
SortingVerified both:
The returned timestamps and frontend rows reorder correctly. 
PaginationVerified:
UI states and responsiveness
|
udaycodespace
pushed a commit
that referenced
this pull request
Jul 18, 2026
Merge PR #17 from Jidnyasa-P Add a secure admin payment dashboard with secure payment record management for administrators. Highlights: - Add admin payment records dashboard - Sanitize payment data returned by the API - Support search, filtering, sorting, and pagination - Display payment summary metrics - Add responsive desktop and mobile layouts - Add accessible transaction details dialog - Protect endpoints with existing admin authentication and authorization - Add API documentation and testing guidance - Verify implementation with backend validation, production build, authorization checks, and feature testing Closes #17. P.S. While merging PR #16, I accidentally used this PR's merge title and description. So now PR #16 has PR #17's merge message, and PR #17 gets PR #16's. They basically swapped identities.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds a secure admin activity-log viewer with search, filters, sorting, pagination, responsive layouts, and safe loading and error states.
Fixes #8
Backend changes
Added a protected admin activity-log endpoint
Added query support for:
Added pagination metadata
Added validation for filter and pagination values
Escaped search input before constructing regular expressions
Added optional IP address and user-agent fields
Sanitized the returned activity-log data
Prevented passwords, tokens, payment information, and other sensitive fields from being returned
Frontend changes
Documentation
docs/admin-activity-logs.mdTesting
Screenshots