Skip to content

WIP: Bug fixes#412

Open
laurenp-2 wants to merge 3 commits intomainfrom
bug_fixes
Open

WIP: Bug fixes#412
laurenp-2 wants to merge 3 commits intomainfrom
bug_fixes

Conversation

@laurenp-2
Copy link
Copy Markdown

@laurenp-2 laurenp-2 commented Nov 17, 2025

Summary

This pull request addresses several bugs and feature enhancements throughout the website.

  • implement clickable map pins
  • debug overflowing apartment preview on map
  • prompt to add missing apartment on search page
  • prompt users to submit a review when submitting a new apartment

Test Plan

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Lauren Pothuru seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@dti-github-bot
Copy link
Copy Markdown
Member

[diff-counting] Significant lines: 84.

@parsa-tehranipoor
Copy link
Copy Markdown
Contributor

This is a pretty neatly written PR. I think you did a really good job of writing your code in sync with the existing files and you mimicked the coding style and naming conventions very well! I think you did a great job here, especially for a short PR.

@NigelTatem
Copy link
Copy Markdown

For some reason I can't leave this as a review so I'll just leave it as a comment:

This is a really clean PR. The changes feel well integrated with the existing structure of the project, nothing feels out of place or overly custom.

I especially appreciated how focused the changes were. Each feature and bug fix is scoped clearly, and the implementation is straightforward without introducing unnecessary complexity. It’s easy to trace what was added and why, and passes my 'review'

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.

5 participants