Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PIMS-2220 Project Properties Map #2863

Merged
merged 6 commits into from
Nov 14, 2024
Merged

PIMS-2220 Project Properties Map #2863

merged 6 commits into from
Nov 14, 2024

Conversation

dbarkowsky
Copy link
Collaborator

@dbarkowsky dbarkowsky commented Nov 13, 2024

🎯 Summary

PIMS-2220

Changes

  • Reworked the cluster popup so it will centre correctly no matter what size of map it's on
  • Added additional configuration options to the parcel map
  • Added a parcel map to the project details page
  • Added parcel boundary details to maps by default
  • PID/Address values in the disposal properties tables are now links to the properties
  • Fixed an issue with the PropertyRow not showing the PIN correctly if the property had a PID with value 0.

Testing

  • Navigate to a project with associated properties
  • See that the map is present and behaves as expected. Should have a cluster popup and some zoom/drag controls
  • Check the main map, property add map, and property details map to see that they also still work as expected.

🔰 Checklist

  • I have read and agree with the following checklist and am following the guidelines in our Code of Conduct document.
  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation where required.
  • I have tested my changes to the best of my ability.
  • My changes generate no new warnings.

@dbarkowsky dbarkowsky self-assigned this Nov 13, 2024
@github-actions github-actions bot added the React label Nov 13, 2024
@dbarkowsky dbarkowsky changed the title PIMS-2220 Project Map PIMS-2220 Project Properties Map Nov 13, 2024
Copy link

🚀 Deployment Information

The React APP Image has been built with the tag: 2863. Please make sure to utilize this specific tag when promoting these changes to the TEST and PROD environments during the APP deployment. For more updates please monitor Image Tags Page on Wiki.

Copy link
Collaborator

@Sharala-Perumal Sharala-Perumal left a comment

Choose a reason for hiding this comment

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

Works perfectly. The map shows up under the Disposal Properties section with Zoom in and out controls. The cluster icon and the popup works fine.

@dbarkowsky dbarkowsky merged commit 3ecdadc into main Nov 14, 2024
8 checks passed
@dbarkowsky dbarkowsky deleted the PIMS-2220-Project-Map branch November 14, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants