Skip to content

Conversation

@younocode
Copy link
Contributor

No description provided.

@younocode younocode requested a review from Copilot October 20, 2025 11:07
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 enhances the base delete event to support permanent deletion by adding a permanent flag that bypasses trash creation when set to true.

  • Added permanent parameter to base delete events to distinguish between soft and permanent deletions
  • Modified the trash listener to skip trash creation for permanent deletions
  • Updated the event factory and interceptor to properly handle the permanent deletion flag

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
apps/nestjs-backend/src/features/base/base.controller.ts Added event emission and permanent flag to permanent delete endpoint
apps/nestjs-backend/src/event-emitter/listeners/trash.listener.ts Added logic to skip trash creation for permanent deletions
apps/nestjs-backend/src/event-emitter/interceptor/event.Interceptor.ts Separated BASE_DELETE event handling from other base events
apps/nestjs-backend/src/event-emitter/events/base/base.event.ts Updated event payload structure to include permanent flag

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@younocode younocode requested a review from boris-w October 21, 2025 05:59
@younocode younocode force-pushed the fix/base-delete-handle branch from 67f73be to 1b9412e Compare October 21, 2025 06:00
@younocode younocode force-pushed the fix/base-delete-handle branch from 1b9412e to 53ba02a Compare October 23, 2025 13:46
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.

1 participant