Skip to content

Conversation

@ryanslatten
Copy link
Contributor

@ryanslatten ryanslatten commented Jul 9, 2025

This PR is a starting point for allowing observations that do not have geometry data. It adds a new checkbox option when creating/editing an observation to not send geometry data and will default to a placeholder.

This is a draft pr and should not be merged in yet. Please recommend changes that you think might be useful for this feature.

Option to disable geometry
Screenshot 2025-07-09 at 4 25 40 PM

Observation view without geometry
Screenshot 2025-07-09 at 4 26 28 PM

Future Action Items:

  • Add filtering for no Geometry after this pr is merged Update and Fix Observation Filtering - MAGE-1501 #299
  • Hide observations with no geometry from map and on the panel if the filter is not set
  • Update Events so that Admins can set noGeometry as a default property for new observations within the event

@ryanslatten ryanslatten marked this pull request as draft July 9, 2025 20:27
@ryanslatten ryanslatten marked this pull request as ready for review July 10, 2025 23:34
jmckomni
jmckomni previously approved these changes Jul 15, 2025
cwerthomni
cwerthomni previously approved these changes Jul 16, 2025
jmckomni
jmckomni previously approved these changes Jul 16, 2025
@ryanslatten ryanslatten dismissed stale reviews from jmckomni and cwerthomni via bf7aa12 August 5, 2025 14:19
@ryanslatten ryanslatten merged commit 0355723 into develop Aug 5, 2025
10 checks passed
@ryanslatten ryanslatten deleted the no-geometry-observations branch August 5, 2025 14:25
@ryanslatten ryanslatten restored the no-geometry-observations branch August 5, 2025 14:25
cwerthomni added a commit that referenced this pull request Aug 8, 2025
* update prod dockerfile and github actions

* Adds User and Form Selection to Filtering Options. Also Fixes the Time Based Filtering.

* Add Comments and a New Type

* Fix Variables

* added CODEOWNERS file.

* Fix Errors found by CJ

* KMZ importing updated to support image uploads (#298)

Improve KMZ file importing where observations with included images are not displayed properly

* moved CODEOWNERS to root of project.

* Homepage updated for support for smaller screens (#303)

* initial commit

* fix title styling

* updated css names

* Publish containers as packages (#304)

* update action to deploy container artifact

* update yml and action name

* Production Container action

* update containers and scripts

* update dev dockerfile

* update dockerfile

---------

Co-authored-by: Clinton Werth <[email protected]>

* Prod container action (#305)

* new docker file and update prod container action

* down to one dockerfile

---------

Co-authored-by: Clinton Werth <[email protected]>
Co-authored-by: Clinton Werth <[email protected]>

* changed time format option from '30 seconds' to 'Relative'

* updated CODEOWNERS

* MOCKUP: Adding support for observations without geometry (#300)

Update the Observation logic to add a noGeometry flag.

* Admin Teams dashboard component upgrade from AngularJS to Angular 14 (#306)

Updates the Teams component to Angular 14

---------

Co-authored-by: Clinton Werth <[email protected]>

* authorization component now submits access code when user hits enter.

* Initial Commit, ts and html changes

* Hook Up Module

* fix modules

* Remove and Revert

* fixes

* directive

* include module

* Pushing latest changes

* Final Commit

* fix css

* Remove DS Store Files

* Make HTML Safe Again

* unknown device

* fix package log

* Updates for 6.4.0-beta.2 release (#311)

Co-authored-by: Clinton Werth <[email protected]>

---------

Co-authored-by: Clinton Werth <[email protected]>
Co-authored-by: Ryan Carpenter <[email protected]>
Co-authored-by: John McKeever <[email protected]>
Co-authored-by: jmckomni <[email protected]>
Co-authored-by: Ryan <[email protected]>
Co-authored-by: Clinton Werth <[email protected]>
Co-authored-by: MilliannaMiodori <[email protected]>
@ryanslatten ryanslatten deleted the no-geometry-observations branch December 11, 2025 18:54
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.

4 participants