Skip to content

Fix issues #129, #130, #94, #106, #105, #126#137

Open
Snehil208001 wants to merge 1 commit intojaygajera17:master2from
Snehil208001:fix-issues-129-130-94-106-105-126
Open

Fix issues #129, #130, #94, #106, #105, #126#137
Snehil208001 wants to merge 1 commit intojaygajera17:master2from
Snehil208001:fix-issues-129-130-94-106-105-126

Conversation

@Snehil208001
Copy link
Copy Markdown

@Snehil208001 Snehil208001 commented Mar 9, 2026

Additional: AdminDataLoader for BCrypt admin password, fix SecurityConfiguration UserDetailsService

…a17#106, jaygajera17#105, jaygajera17#126

- jaygajera17#129: Add delete/toggleActive functionality for customers (AdminCustomerController)
- jaygajera17#130: Add isActive boolean flag in User model with soft delete and security checks
- jaygajera17#94: Fix Update Product Details (AdminProductController, productsUpdate.jsp)
- jaygajera17#106: Fix misaligned login button on mobile (adminlogin.jsp CSS)
- jaygajera17#105: Add Dockerfile and docker-compose.yml for easier setup
- jaygajera17#126: Split Admin controller into AdminController, AdminCategoryController, AdminProductController, AdminCustomerController

Additional: AdminDataLoader for BCrypt admin password, fix SecurityConfiguration UserDetailsService
Made-with: Cursor
@Snehil208001 Snehil208001 force-pushed the fix-issues-129-130-94-106-105-126 branch from ec40169 to 2fda2e0 Compare March 9, 2026 08:38
@Snehil208001
Copy link
Copy Markdown
Author

Testing instructions

Setup:

  1. Configure application.properties with your MySQL credentials (db.url, db.username, db.password)
  2. Run basedata.sql on your database, or use insert_admin.sql if the customer table is empty
  3. If admin login fails, run fix_admin_password.sql to set BCrypt hash for admin/123

Admin login: admin / 123
User login: lisa / 765

Docker: docker-compose up from project root (requires MySQL running or use the compose MySQL service)

Verified: All Maven tests pass locally.

@Snehil208001
Copy link
Copy Markdown
Author

Key files changed

Issue Files
#129, #130 AdminCustomerController.java, displayCustomers.jsp, User.java
#94 AdminProductController.java, productsUpdate.jsp
#106 adminlogin.jsp
#105 Dockerfile, docker-compose.yml, .dockerignore
#126 AdminCategoryController.java, AdminProductController.java, AdminCustomerController.java (split from AdminController)
Security SecurityConfiguration.java, AdminDataLoader.java

@jaygajera17
Copy link
Copy Markdown
Owner

Thanks @Snehil208001 , I'll check this

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.

2 participants