Skip to content

Add shared Swagger DTOs, audit diff helper, activity feed mapper, evidence validation - #511

Merged
KuchiMercy merged 1 commit into
StayLitCodes:mainfrom
benzeneCLI:feat/issues-405-413-488-489
Jul 28, 2026
Merged

Add shared Swagger DTOs, audit diff helper, activity feed mapper, evidence validation#511
KuchiMercy merged 1 commit into
StayLitCodes:mainfrom
benzeneCLI:feat/issues-405-413-488-489

Conversation

@benzeneCLI

Copy link
Copy Markdown
Contributor

A lot of the ground for these four issues already exists (Swagger is already configured in main.ts, AdminAuditLogService already has create/findAll with filters+pagination, and analytics.service.ts already has summary/volume/users/disputes/top-users with in-memory TTL caching). This PR adds the narrower pieces that were still missing, each as its own new file:

  • config::swagger-responses.dto: shared ErrorResponseDto/PaginationMetaDto so controllers can reference one consistent @ApiResponse shape instead of each hand-rolling it. Decorating every controller/handler per the full acceptance criteria is a much larger follow-up.
  • admin::audit-diff.helper: buildStateDiff computing a previousState/newState diff to embed in the existing audit log metadata field, plus the missing piece toward a per-escrow history endpoint.
  • admin::analytics-activity.helper: toRecentActivityFeed mapping AdminAuditLog rows to the shape a GET /admin/analytics/activity route needs — the one endpoint from the issue not already present alongside summary/volume/users/disputes/top-users.
  • escrow::evidence-file-validation: size + MIME-type validators for evidence uploads, which EvidenceFileMetadataDto currently doesn't enforce.

Closes #405
Closes #413
Closes #488
Closes #489

@drips-wave

drips-wave Bot commented Jul 28, 2026

Copy link
Copy Markdown

@benzeneCLI Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@KuchiMercy
KuchiMercy merged commit ca5db19 into StayLitCodes:main Jul 28, 2026
1 check 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

2 participants