Skip to content

Saikat dash server.js updated - #124

Open
SaikatDash wants to merge 2 commits into
hitesh-kumar123:mainfrom
SaikatDash:SaikatDash-server.js-updated
Open

Saikat dash server.js updated#124
SaikatDash wants to merge 2 commits into
hitesh-kumar123:mainfrom
SaikatDash:SaikatDash-server.js-updated

Conversation

@SaikatDash

Copy link
Copy Markdown

name: "📦 Pull Request"
about: Submit changes for review
title: "PR: Update server configuration to support location-based search API"
labels: enhancement
assignees: ""

📌 Linked Issue


🛠 Changes Made

  • Updated
    • Registered the property search routes (/api/properties/search) in server.js.
    • Ensured search endpoints are properly exposed via the main Express app.
  • Improved
    • API routing structure to support scalable feature additions.
    • Consistent middleware order for request handling.

🧪 Testing

  • Ran unit tests (npm test)
  • Tested manually:
    • Test case 1:
      Steps: Start the backend server
      Expected Result: Server runs without errors.
    • Test case 2:
      Steps: Send a GET request to /api/properties/search
      Expected Result: API responds successfully with search results.

📸 UI Changes (if applicable)

Before After
N/A N/A

📝 Documentation Updates

  • Updated README/docs
  • Added inline comments in server.js 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

  • No breaking changes introduced.
  • This PR only wires new routes; business logic remains in controllers.
  • Fully compatible with existing middleware and environment configuration.

@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 217a7cf
🔍 Latest deploy log https://app.netlify.com/projects/smartrentsystem/deploys/69674af7992a4900083b8998

@hitesh-kumar123

Copy link
Copy Markdown
Owner

This PR is invalid because you did not assign an issue. So i canceled the PR

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