Thanks to visit codestin.com
Credit goes to github.com

Skip to content

PIMS-339 Map Zoom When Searching for PID#1987

Merged
TaylorFries merged 23 commits intomainfrom
PIMS-399-pid-search-zoom
Jan 9, 2024
Merged

PIMS-339 Map Zoom When Searching for PID#1987
TaylorFries merged 23 commits intomainfrom
PIMS-399-pid-search-zoom

Conversation

@TaylorFries
Copy link
Collaborator

@TaylorFries TaylorFries commented Dec 19, 2023

🎯 Summary

PIMS-399

Updating the "Add land" form such that when searching for a PID that already exists in PIMS information is updated and the map zooms to the location of the found PID.

Testing

*** This ticket was completed with another PR (can't figure out which one exactly) But the 'bug' is fixed. This PR is now just for changing the styled-components to a scss file.

To test please load PIMS, and navigate to "Submit property" and ensure that the right hand menu looks and operates as it should.

🔰 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.

@github-actions
Copy link

🚀 Deployment Information

The APP Image has been built with the tag: 1987. 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.

@qlty-cloud-legacy
Copy link

qlty-cloud-legacy bot commented Dec 19, 2023

Code Climate has analyzed commit e06f942 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 0.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 55.8%.

View more on Code Climate.

@TaylorFries TaylorFries self-assigned this Jan 3, 2024
Copy link
Collaborator

@dbarkowsky dbarkowsky left a comment

Choose a reason for hiding this comment

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

The original bug seems to pop up and down today.
Based on our chat earlier, this might be a long-term issue.
I'm thinking we get these changes in for now, and if anyone has additional changes in the future that specifically address the bug, that can be its own PR.

@TaylorFries TaylorFries merged commit 4e25920 into main Jan 9, 2024
@TaylorFries TaylorFries deleted the PIMS-399-pid-search-zoom branch January 9, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants