Skip to content

Feat/real world market locations#77

Open
Aharshi3614 wants to merge 7 commits into
jpdevhub:mainfrom
Aharshi3614:feat/real-world-market-locations
Open

Feat/real world market locations#77
Aharshi3614 wants to merge 7 commits into
jpdevhub:mainfrom
Aharshi3614:feat/real-world-market-locations

Conversation

@Aharshi3614
Copy link
Copy Markdown
Contributor

Closes #75

What this does

Adds a new /api/v1/maps/markets/live endpoint that queries the OpenStreetMap
Overpass API to fetch real fish markets, seafood shops, and marketplaces near
a user's GPS location.

New endpoint

GET /api/v1/maps/markets/live?lat={lat}&lng={lng}&radius={meters}

Response:
{
"success": true,
"source": "openstreetmap",
"count": 3,
"markets": [
{ "id": 123, "name": "Fish Market", "lat": 12.34, "lng": 56.78, "vendors": 1, "source": "openstreetmap" }
]
}

Implementation

  • Created backend/markets.py with Overpass API integration
  • Queries nodes tagged shop=seafood, shop=fish, and fish/seafood marketplaces
  • Converts lat/lng + radius to bounding box for Overpass query
  • Normalizes response to match existing MarketMapPage.tsx format
  • Graceful fallback on timeout or API failure
  • Registered router in main.py

Why OpenStreetMap over Google Places

  • Free with no API key or billing required
  • Global coverage including rural fish markets in India
  • Matches the acceptance criteria in the issue

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 4, 2026

@Aharshi3614 is attempting to deploy a commit to the karan3431's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

🎉 Thank you for your Pull Request! We're thrilled to have your contribution to FreshScan AI.

Before we review, please make sure you have:

  • Followed the CONTRIBUTING.md guidelines.
  • Ensured all automated CI checks (linting, tests) are passing.
  • Checked that your commit messages follow the Conventional Commits format.

A maintainer will review your code as soon as possible!

@jpdevhub
Copy link
Copy Markdown
Owner

jpdevhub commented Jun 6, 2026

Can you just upload a ss or video to show this feature in working state to make it merge shortly as i am tired it takes 4+ hr to debug and settle the issue of the onnx update today. Dont forget to pull the latest changes.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
fresh-scan-ai Ready Ready Preview, Comment Jun 6, 2026 10:01am

@Aharshi3614
Copy link
Copy Markdown
Contributor Author

image Here is the ss of it working!

@jpdevhub
Copy link
Copy Markdown
Owner

jpdevhub commented Jun 6, 2026

pull the latest changes and try to fix the backend lint test, if not dont take the pressure much i will be checking it tonight.

@jpdevhub
Copy link
Copy Markdown
Owner

jpdevhub commented Jun 7, 2026

The failing job is the Lint step in the CI workflow. The error shows ESLint found 2 linting errors:

  1. Unused variable math - This variable is defined but never used in your code
  2. Missing semicolon - One or more statements are missing semicolons

Or run this command locally to identify and fix the issues:

npm run lint -- --fix

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.

Integrate real-world fish market locations using public APIs

2 participants