Skip to content

Saikat dash updating the property routes,js#122

Open
SaikatDash wants to merge 2 commits into
hitesh-kumar123:mainfrom
SaikatDash:SaikatDash-updating-the-propertyRoutes,js
Open

Saikat dash updating the property routes,js#122
SaikatDash wants to merge 2 commits into
hitesh-kumar123:mainfrom
SaikatDash:SaikatDash-updating-the-propertyRoutes,js

Conversation

@SaikatDash

Copy link
Copy Markdown

name: "📦 Pull Request"
about: Submit changes for review
title: "PR: Update property routes for location-based search"
labels: enhancement
assignees: ""

📌 Linked Issue


🛠 Changes Made

  • Added
    • New /api/properties/search GET route to support location-based filtering.
  • Updated
    • propertyRoutes.js to connect the search endpoint with the controller.
  • Improved
    • Route structure to support scalable search parameters (city, state, latitude, longitude).

🧪 Testing

  • Ran unit tests (npm test)
  • Tested manually:
    • Test case 1:
      Steps: Send GET request to /api/properties/search?city=Kolkata
      Expected Result: Properties from Kolkata are returned.
    • Test case 2:
      Steps: Send GET request to /api/properties/search?lat=22.57&lng=88.36
      Expected Result: Nearby properties are returned using geospatial query.

--

📝 Documentation Updates

  • Updated README/docs
  • Added route-level comments for clarity

✅ Checklist

  • Created a new branch for PR
  • Have starred the repository
  • Follows JavaScript Styleguide
  • No console warnings/errors
  • Commit messages follow Git Guidelines

💡 Additional Notes

  • This PR only updates routing logic.
  • Business logic is handled inside the controller.
  • Fully backward compatible with existing routes.

Added a new route to fetch all properties without filtering.
@netlify

netlify Bot commented Jan 14, 2026

Copy link
Copy Markdown

Deploy Preview for smartrentsystem canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 7833a97
🔍 Latest deploy log https://app.netlify.com/projects/smartrentsystem/deploys/69674a1bf2e37a00083e89a6

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.

Feat: [Brief Description] In Search Bar adding my location first as per my profile would be great for searching in my area

2 participants