Skip to content

Fix/architecture#167

Open
navin-oss wants to merge 3 commits intoNitya-003:mainfrom
navin-oss:fix/architecture
Open

Fix/architecture#167
navin-oss wants to merge 3 commits intoNitya-003:mainfrom
navin-oss:fix/architecture

Conversation

@navin-oss
Copy link
Contributor

Fixed the backend data store confusion by removing the deprecated in-memory store:

Changes Made:

Analysis:
The backend had two data stores:

  1. MongoDB via Batch.js - The primary data store used by all routes in server.js
  2. In-memory store in BatchStore.js - An unused Map-based store initialized with sample data that was never imported anywhere in the codebase

The BatchStore.js file contained:

  • A Map-based in-memory storage with sample batch data
  • A counter for generating batch IDs (e.g., CROP-2024-001)
  • Was never imported or used by any route/controller

All routes in server.js correctly use MongoDB via the Batch model (see lines 320-536), so removing this unused file eliminates the confusion without affecting functionality.

Branch pushed: fix/architecture
Commit: 59b2853 - fix: remove deprecated BatchStore.js to eliminate data store confusion

@netlify
Copy link

netlify bot commented Mar 6, 2026

Deploy Preview for roaring-ganache-c7465f failed. Why did it fail? →

Name Link
🔨 Latest commit e59c874
🔍 Latest deploy log https://app.netlify.com/projects/roaring-ganache-c7465f/deploys/69ad5b53869e9800081104ed

@vedhapprakashni
Copy link
Collaborator

pls resolve conflicts @navin-oss .which issue does this close?

Copy link
Owner

@Nitya-003 Nitya-003 left a comment

Choose a reason for hiding this comment

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

@navin-oss Please mention the issue number and resolve the conflicts.

Image

@vedhapprakashni
Copy link
Collaborator

@navin-oss pls resolve conflicts and site the issue it addresses.apertre is ending soon.

Copy link
Owner

@Nitya-003 Nitya-003 left a comment

Choose a reason for hiding this comment

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

@navin-oss Closes #153

Image

@vedhapprakashni vedhapprakashni linked an issue Mar 8, 2026 that may be closed by this pull request
@navin-oss
Copy link
Contributor Author

Hi @Nitya-003,
I attempted to resolve the merge conflicts locally, but I'm still running into issues while syncing my branch with upstream/main. The conflicts seem to involve backend/server.js, and I’m currently unable to resolve them correctly from my side.

@Nitya-003
Copy link
Owner

@navin-oss You can have your time. and check conflicts on your other PRs too.

@vedhapprakashni
Copy link
Collaborator

@navin-oss pls check this asap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inconsistent Persistence Layers*

3 participants