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

CE-199-Refactoring of edit and geocoding functionality #181

Merged
merged 6 commits into from
Oct 30, 2023
Merged

Conversation

barrfalk
Copy link
Contributor

@barrfalk barrfalk commented Oct 28, 2023

Description

A number of changes were made here to improve the edit complaint location and geocoding functionality.

  1. Refactored variable names to improve readability
  2. Reduced minScore from 2 to 1 on geocoding since the geocoder was being overly picky on locality names and addresses.
  3. Complaint coordinates are now displayed on edit screen
  4. Updates the lat/lng values when the coordinates change so that the map reflects the latest and greatest coordinates

Fixes # CE-199

How Has This Been Tested?

  • Verified x and y coordinates are displayed on the edit screen
  • Verified saves to x and y coordinates are reflected on details map
  • Verified that "Pilot Mountain" and "Pilot Bay" which were previously not found by the geocoder are now found.
  • Verified that when the community changes the location on the map updates.
  • Verified that a user can create a complaint without a coordinate and that the map is centered based on the community name.
  • Verified that editing a complaint without coordinates and filling in the coordinates will correctly display the marker on the map at the entered coordinates.

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Further comments


Thanks for the PR!

Any successful deployments (not always required) will be available below.
Backend available
Frontend available

Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

A number of changes were made here to improve the edit complaint location and geocoding functionality.

1. Refactored variable names to improve readability
2. Reduced minScore from 2 to 1 on geocoding since the geocoder was being overly picky on locality names and addresses.
3. Complaint coordinates are now displayed on edit screen
4. Updates the lat/lng values when the coordinates change so that the map reflects the latest and greatest coordinates
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 19:46 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 19:46 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 19:46 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 19:46 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 20:11 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 20:11 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 20:11 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 20:11 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 20:19 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 20:19 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 20:19 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 20:19 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 21:14 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 21:14 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 21:14 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 28, 2023 21:14 — with GitHub Actions Inactive
@barrfalk barrfalk marked this pull request as ready for review October 28, 2023 22:22
@barrfalk barrfalk enabled auto-merge October 28, 2023 22:23
@barrfalk barrfalk temporarily deployed to dev October 30, 2023 18:35 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 30, 2023 18:35 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 30, 2023 18:35 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 30, 2023 18:35 — with GitHub Actions Inactive
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@barrfalk barrfalk temporarily deployed to dev October 30, 2023 19:14 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 30, 2023 19:14 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 30, 2023 19:14 — with GitHub Actions Inactive
@barrfalk barrfalk temporarily deployed to dev October 30, 2023 19:14 — with GitHub Actions Inactive
@barrfalk barrfalk added this pull request to the merge queue Oct 30, 2023
Merged via the queue into main with commit 50ae17a Oct 30, 2023
15 of 16 checks passed
@barrfalk barrfalk deleted the bug/CE-199 branch October 30, 2023 19:59
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.

2 participants