Skip to content

Adding zone export to GeoJSON, added versioning and hash updates#1312

Merged
Azgaar merged 4 commits into
Azgaar:masterfrom
JoeMcMahon87:export-zones-geojson
Feb 19, 2026
Merged

Adding zone export to GeoJSON, added versioning and hash updates#1312
Azgaar merged 4 commits into
Azgaar:masterfrom
JoeMcMahon87:export-zones-geojson

Conversation

@JoeMcMahon87
Copy link
Copy Markdown
Contributor

Description

Adding zone export to GeoJSON. Works like Rivers and Routes exports. New end to end testing added to cover this feature.

Type of change

  • Bug fix
  • New feature
  • Refactoring / style
  • Documentation update / chore
  • Other (please describe)

Versioning

  • Version is updated
  • Changed files hash is updated

@JoeMcMahon87
Copy link
Copy Markdown
Contributor Author

Create a new, cleaner version, minimal file changes. npm run build and npm run lint run clean for my changes (still seeing some lint errors in other files)

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 11, 2026

Deploy Preview for afmg ready!

Name Link
🔨 Latest commit 9883aa6
🔍 Latest deploy log https://app.netlify.com/projects/afmg/deploys/69976afb3deb5f0009dcc5ab
😎 Deploy Preview https://deploy-preview-1312--afmg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread tests/e2e/zones-export.spec.ts Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new GeoJSON export option for map zones (similar to existing Routes/Rivers exports), wires it into the UI, and bumps versioning + cache-busting hashes. Includes a new Playwright e2e spec intended to validate the zones GeoJSON export.

Changes:

  • Add saveGeoJsonZones() implementation to export zones as GeoJSON and expose it via a new “zones” export button.
  • Update versioning.js version + update notes, and refresh export.js hash in src/index.html.
  • Add tests/e2e/zones-export.spec.ts covering zone GeoJSON export behavior.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
public/modules/io/export.js Adds saveGeoJsonZones() GeoJSON exporter for zones
src/index.html Adds “zones” GeoJSON export button and bumps export.js cache-busting hash
public/versioning.js Bumps VERSION and adds update note entry
tests/e2e/zones-export.spec.ts Adds new Playwright tests around zone GeoJSON export
package-lock.json Lockfile metadata updates

Comment thread public/modules/io/export.js
Comment thread tests/e2e/zones-export.spec.ts Outdated
Comment thread tests/e2e/zones-export.spec.ts Outdated
Comment thread tests/e2e/zones-export.spec.ts Outdated
@JoeMcMahon87
Copy link
Copy Markdown
Contributor Author

Ok, I think this cleaned it up (and I think I get the process now). Appreciate your patience

@Azgaar
Copy link
Copy Markdown
Owner

Azgaar commented Feb 19, 2026

Thanks, it looks really good.

@Azgaar Azgaar merged commit 0ff0311 into Azgaar:master Feb 19, 2026
7 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