Skip to content

Reina create interactive map showing location of projects backend#1342

Closed
ReinaT5678 wants to merge 8 commits intodevelopmentfrom
reina-create-interactive-map-showing-location-of-projects-backend
Closed

Reina create interactive map showing location of projects backend#1342
ReinaT5678 wants to merge 8 commits intodevelopmentfrom
reina-create-interactive-map-showing-location-of-projects-backend

Conversation

@ReinaT5678
Copy link
Contributor

@ReinaT5678 ReinaT5678 commented Apr 17, 2025 •
edited
Loading

Description

Backend for frontend PR #3432
Please checkout frontend PR for more details.

Related PRS (if any):

To test this backend PR you need to checkout the #XXX frontend PR.
…

Main changes explained:

  • Created a new Schema for organization location details
  • Made new routes for retrieving org data from the database
    …

How to test:

  1. check into current branch
  2. do npm install and npm start to run this PR locally
  3. Check out frontend for further steps

This was referenced Apr 21, 2025
Closed
Closed
@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label May 9, 2025
shashank-madan
shashank-madan reviewed Jul 9, 2025
View reviewed changes
Copy link
Contributor

@shashank-madan shashank-madan left a comment

Choose a reason for hiding this comment

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

Changes work as expected. Reviewed with screenshots in frontend PR #3432. Review comments

vamsidharpanithi
vamsidharpanithi reviewed Jul 13, 2025
View reviewed changes
Copy link
Contributor

@vamsidharpanithi vamsidharpanithi left a comment

Choose a reason for hiding this comment

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

Checked out and tested this backend along with the related frontend PR. The new schema for organization location details and the routes for retrieving organization data are working as expected. The API returns the correct location data, which integrates properly with the interactive map on the frontend. Verified that the data supports marker display and filtering by date without any issues. The backend functionality appears to be implemented correctly.

adithya-6101
adithya-6101 approved these changes Jul 16, 2025
View reviewed changes
Copy link
Contributor

@adithya-6101 adithya-6101 left a comment

Choose a reason for hiding this comment

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

Tested this backend along with the related frontend PR and everything works well. The interactive map loads correctly, the API also returns the correct location data and the dots are clickable. Looks great approving this!

PR-3432+1342-2 PR-3432+1342-1

Neeraj-Kondaveeti
Neeraj-Kondaveeti approved these changes Jul 16, 2025
View reviewed changes
Copy link
Contributor

@Neeraj-Kondaveeti Neeraj-Kondaveeti left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Tested this backend PR with the related frontend changes (#3432). The organization location data is loading correctly and supports the map feature as intended. Everything looks good—approving the PR.

3432(1)
3432(2)

uhakruthi1
uhakruthi1 approved these changes Jul 26, 2025
View reviewed changes
Copy link
Contributor

@uhakruthi1 uhakruthi1 left a comment

Choose a reason for hiding this comment

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

Tested this backend together with the corresponding frontend PR — everything functions smoothly. The interactive map renders properly, the API delivers accurate location data, and the dots are fully clickable. Everything looks solid, approving it!
Screenshot 2025-07-26 at 10 30 46 AM
Screenshot 2025-07-26 at 10 31 05 AM

harshavarma29
harshavarma29 approved these changes Jul 27, 2025
View reviewed changes
Copy link
Contributor

@harshavarma29 harshavarma29 left a comment

Choose a reason for hiding this comment

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

I have tested the related Frontend PR, and the backend functionality is working as expected. However, I noticed a few minor issues on the frontend side and have left some comments in the Frontend PR for review.

@ReinaT5678 ReinaT5678 mentioned this pull request Aug 17, 2025
Closed
@SallaguntaRaahul
Copy link

SallaguntaRaahul commented Aug 22, 2025

The corresponding frontend PR #3915 works as expected

aseemdeshmukh
aseemdeshmukh approved these changes Aug 23, 2025
View reviewed changes
Copy link

@aseemdeshmukh aseemdeshmukh left a comment

Choose a reason for hiding this comment

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

Approved on PR3915, everything works perfect!

vishnu-ing
vishnu-ing reviewed Aug 23, 2025
View reviewed changes
Copy link
Contributor

@vishnu-ing vishnu-ing left a comment

Choose a reason for hiding this comment

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

The file changes and functionality has been reviewed. I have added some comments in the FE PR-3915 for some changes.

@one-community one-community added Do Not Review Do not review or look at code without full context Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. and removed High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible labels Sep 1, 2025
@jaydeep138 jaydeep138 mentioned this pull request Oct 3, 2025
Merged
@one-community one-community mentioned this pull request Oct 19, 2025
Open
@maithili20 maithili20 mentioned this pull request Feb 21, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vamsidharpanithi vamsidharpanithi vamsidharpanithi left review comments

@harshavarma29 harshavarma29 harshavarma29 approved these changes

@Neeraj-Kondaveeti Neeraj-Kondaveeti Neeraj-Kondaveeti approved these changes

@adithya-6101 adithya-6101 adithya-6101 approved these changes

@aseemdeshmukh aseemdeshmukh aseemdeshmukh approved these changes

@uhakruthi1 uhakruthi1 uhakruthi1 approved these changes

+2 more reviewers

@shashank-madan shashank-madan shashank-madan left review comments

@vishnu-ing vishnu-ing vishnu-ing left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Do Not Review Do not review or look at code without full context Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.