Skip to content

Add delete customer functionality in admin panel#131

Open
golukumar2156 wants to merge 1 commit intojaygajera17:master2from
golukumar2156:master2
Open

Add delete customer functionality in admin panel#131
golukumar2156 wants to merge 1 commit intojaygajera17:master2from
golukumar2156:master2

Conversation

@golukumar2156
Copy link
Copy Markdown

Description

This PR adds the missing delete functionality for customers in the admin panel.

Changes

  • Added delete customer endpoint in AdminController
  • Added delete link/button in customers JSP page

Root Cause

The delete option was visible in the UI but no backend/view logic was implemented.

Fixes #129

@golukumar2156
Copy link
Copy Markdown
Author

Hi maintainer 👋
The failing check is from the Labeler GitHub Action.
It seems unrelated to the code changes and likely due to labeler configuration.
Please let me know if any changes are needed from my side.

Copy link
Copy Markdown

@abdurraffaysyed abdurraffaysyed left a comment

Choose a reason for hiding this comment

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

Hi @golukumar2156,
Instead of hard deleting the customers, I was thinking of updating their status to inActive. With this approach if the same user comes in later on then instead of adding their details all over again we can simply change their Active status. What do you say?

@golukumar2156
Copy link
Copy Markdown
Author

golukumar2156 commented Jan 17, 2026 via email

Copy link
Copy Markdown

@abdurraffaysyed abdurraffaysyed left a comment

Choose a reason for hiding this comment

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

@golukumar2156
I have raised an issue for the SoftDelete functionality too. You can have a review of that for a better understanding.

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.

Delete functionality for the Customers is missing

2 participants