Skip to content

Implement LOI property generator for GeoIDs#2484

Open
rfontanarosa wants to merge 15 commits into
masterfrom
rfontanarosa/2438/possibility-to-enable-or-disable-integrated-services-3
Open

Implement LOI property generator for GeoIDs#2484
rfontanarosa wants to merge 15 commits into
masterfrom
rfontanarosa/2438/possibility-to-enable-or-disable-integrated-services-3

Conversation

@rfontanarosa
Copy link
Copy Markdown
Collaborator

@rfontanarosa rfontanarosa commented Apr 3, 2026

closes #2438
closes #1537
closes #2113

Screenshot 2026-05-21 alle 11 58 43

@rfontanarosa rfontanarosa self-assigned this Apr 3, 2026
@rfontanarosa rfontanarosa changed the title Rfontanarosa/2438/possibility to enable or disable integrated services 3 Implement GeoID integration Apr 3, 2026
@rfontanarosa rfontanarosa marked this pull request as ready for review April 16, 2026 13:34
@auto-assign auto-assign Bot requested a review from gino-m April 16, 2026 13:34
Copy link
Copy Markdown
Collaborator

@gino-m gino-m left a comment

Choose a reason for hiding this comment

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

Please also update the PR description to something a bit more self-descriptive, e.g., "Implement property generator for GeoIDs"

Comment thread functions/src/property-generators/geoid.ts Outdated
Comment thread functions/src/property-generators/geoid.ts Outdated
Comment thread functions/src/on-create-loi.ts Outdated
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.58%. Comparing base (ce86519) to head (8693ab8).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2484   +/-   ##
=======================================
  Coverage   62.58%   62.58%           
=======================================
  Files         120      120           
  Lines        2865     2865           
  Branches      425      425           
=======================================
  Hits         1793     1793           
  Misses       1012     1012           
  Partials       60       60           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rfontanarosa rfontanarosa changed the title Implement GeoID integration Implement LOI property generator for GeoIDs Apr 17, 2026
@rfontanarosa rfontanarosa requested a review from gino-m April 17, 2026 07:26
@gino-m
Copy link
Copy Markdown
Collaborator

gino-m commented May 22, 2026

Doesn't WHISP already issue geoIDs? Why do we need a separate generator for GeoIDs?

@jo-spek
Copy link
Copy Markdown
Collaborator

jo-spek commented May 22, 2026

Doesn't WHISP already issue geoIDs? Why do we need a separate generator for GeoIDs?

GeoIDs are supposed to be the ultimate pilar of interoperability. So that a geometry created in Ground can easily be loaded elsewhere without having to export import. If you would generate GeoIDs in Whisp, you would have to manually export-import from Ground to Whisp. We want any newly created geometry to have a GeoID immediately.

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

Labels

None yet

Projects

None yet

4 participants