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

Psp 8635, psp-8636, psp-8638 #4104

Merged
merged 4 commits into from
Jun 12, 2024
Merged

Psp 8635, psp-8636, psp-8638 #4104

merged 4 commits into from
Jun 12, 2024

Conversation

devinleighsmith
Copy link
Collaborator

see commits for story definitions.

@devinleighsmith devinleighsmith added bug Something isn't working hotfix 5.3 labels Jun 11, 2024
@devinleighsmith devinleighsmith self-assigned this Jun 11, 2024
@devinleighsmith devinleighsmith changed the base branch from dev to test June 11, 2024 23:47
@@ -92,8 +92,8 @@ export const PointClusterer: React.FC<React.PropsWithChildren<PointClustererProp
return mapMachine.filePropertyLocations.map(x => {
// The values on the feature are rounded to the 4th decimal. Do the same to the draft points.
return {
lat: parseFloat(x.lat.toFixed(4)),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this was causing some draft markers to be offset from their parent markers at high zoom levels.

@@ -11,7 +11,7 @@
<id>contact</id>
</contact>
<charset>UTF-8</charset>
<numDecimals>4</numDecimals>
<numDecimals>8</numDecimals>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I believe this is the reason the draft markers were being limited to four decimal places - but this restriction only exists on our local geoserver and not the moti one.

Copy link
Contributor

✅ No secrets were detected in the code.

Copy link
Contributor

✅ No secrets were detected in the code.

@asanchezr asanchezr merged commit f7feda2 into bcgov:test Jun 12, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5.3 bug Something isn't working hotfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants