Replace console.log/error with winston logger in approvalRoutes.js

[nishupr]

Bug Description

backend/routes/approvalRoutes.js uses console.log and console.error in 14 places while the rest of the backend uses the centralized Winston logger from utils/logger.js.

Evidence

Running Select-String on backend/routes/approvalRoutes.js reveals 14 console statement instances across:

• Approval request creation → console.log, console.error
• Approval fetch operations → console.error (3 instances)
• Signature addition → console.log, console.error
• Approval cancellation → console.log, console.error
• History and stats → console.error (2 instances)
• Recall/Contamination/Destroy requests → console.error (3 instances)

Impact

• Approval workflow events not captured in logs/combined.log or logs/error.log
• Multi-signature approval failures missed by production monitoring
• Inconsistent logging format across the codebase

Proposed Fix

Import and use the Winston logger in approvalRoutes.js:

• console.log → logger.info
• console.error → logger.error

[github-actions]

🚜 Welcome to the CropChain Ecosystem!

Thank you for opening this issue. Traceability and transparency in agriculture start with great feedback like yours!

Our maintainers will review this shortly. Please make sure you have:

• Used the Issue Template.
• Provided logs or screenshots if reporting a bug.
• Specified if the issue relates to the Smart Contracts, Backend, or Frontend.

🌟 Star the repo to support transparent supply chains!

[nishupr]

hey @Nitya-003 please assign this issue to me.