Skip to content

Allow KMZ filetype uploads for Layers #291

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

Merged
merged 4 commits into from
Jun 25, 2025
Merged

Allow KMZ filetype uploads for Layers #291

merged 4 commits into from
Jun 25, 2025

Conversation

ryanslatten
Copy link
Contributor

This PR allows KMZ filetypes to be uploaded like a KML file. The KMZ file is unzipped before being uploaded.
Screenshot 2025-06-10 at 10 30 27 PM

To test: go to the Admin page and create a new Static Layer. Then attempt to upload a KML, KMZ, and any other file and verify that only KML and KMZ files succeed.

@paulsolt-ofsw
Copy link
Contributor

A few issues:

  1. The KMZ or KML files do not appear in the map preview on the Layers screen after it shows "Created 4 Features from File sample.KMZ"
2025-6-24 No KML data appears on layer map - how to edit
  1. Is there any way to edit multiple KML/KMZ files (remove layers?) - may be a separate backlog item

  2. Minor: If I press upload before loading a file, it appears to load that file immediately, should the upload button be disabled until I have something to upload?

  3. I cannot see the Camera.kmz file. This has styling, but even loading the file as .kml I don't see any points. Previously, @RyCarpenter had seen invalid image icons on the map.

2025-6-24 Cameras KMZ or KML file does not appear on map
  1. Are we loading styling information from the .KMZ file? (Cameras.kmz) should have a custom icon.

  2. Can you mention KML/KMZ in the places that showcase this feature.

2025-6-24 Add KML or KMZ file 2025-6-24 Static KML KMZ file

Copy link
Contributor

@paulsolt-ofsw paulsolt-ofsw left a comment

Choose a reason for hiding this comment

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

See my comment for details. We may need to write new stories to support this feature.

Copy link
Contributor

@paulsolt-ofsw paulsolt-ofsw left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for the fixes.

@ryanslatten ryanslatten merged commit 0c283db into develop Jun 25, 2025
8 checks passed
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.

3 participants