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.

Sign up for GitHub

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

Jump to bottom

Locationless Project Violations #2073

Merged

JoeCohen merged 2 commits into main from jdc-2072-locationless-project-violations

Mar 29, 2024

Member

JoeCohen commented Mar 29, 2024

Prevents ViolationsController#index from throwing an Error if a Project without a Location has Observation(s) with geolocations.
(Fixes Locationless Project Violations #2072.)
Prevents incorrect highlighting of Observation geolocations. The fix described above exposed a bug in which all Observation lat/lon's were incorrectly highlighted as violations in Projects without Location's.

JoeCohen added 2 commits

March 28, 2024 20:11


          Fix Error: ViolationsController#index, nil Location Proj


          Fix incorrect highlighting of lat/lon

739dc3e

JoeCohen linked an issue

that may be closed by this pull request

Locationless Project Violations #2072

Closed

Collaborator

coveralls commented Mar 29, 2024

coverage: 94.401% (+0.001%) from 94.4%
when pulling 739dc3e on jdc-2072-locationless-project-violations
into 2134298 on main.

JoeCohen merged commit a39126f into main

5 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment