KMZ importing updated to support image uploads#298
Merged
Conversation
jmckomni
suggested changes
Jul 9, 2025
Contributor
jmckomni
left a comment
jmckomni
left a comment
There was a problem hiding this comment.
Requested some changes. Reach out to me either in this PR or directly if you have concerns or questions.
jmckomni
approved these changes
Jul 11, 2025
RyCarpenter
approved these changes
Jul 16, 2025
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 <clinton@Clintons-MacBook-Pro.local> * Prod container action (#305) * new docker file and update prod container action * down to one dockerfile --------- Co-authored-by: Clinton Werth <clinton@Clintons-MacBook-Pro.local> Co-authored-by: Clinton Werth <clinton@MacBook-Pro.local> * 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 <clinton@MacBook-Pro.local> * 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 <clinton@MacBook-Pro.local> --------- Co-authored-by: Clinton Werth <clinton@Clintons-MacBook-Pro.local> Co-authored-by: Ryan Carpenter <ryan.carpenter@omnifederal.com> Co-authored-by: John McKeever <johnmckeever@Johns-MacBook-Pro.local> Co-authored-by: jmckomni <john.mckeever@omnifederal.com> Co-authored-by: Ryan <46985385+ryanslatten@users.noreply.github.com> Co-authored-by: Clinton Werth <clinton@MacBook-Pro.local> Co-authored-by: MilliannaMiodori <cozycollectorclub@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR fixes an issue with KMZ uploading where image files included are not being used resulting in a blank image displayed. This also fixes an issue where invalid images would be hidden instead of defaulting to the standard icon.
Example feature layer using features that reference a camera icon included in the KMZ.
To test: Import KMZ files that make use of images that are packaged in the KMZ file. Also ensure that features that reference an invalid image will fallback to the default icon.
A consideration that we may want to implement is to store icons separately in the geojson return value and have features point there to avoid duplication. Another option is to have an icon repository where all non http icons would be stored