Implement LOI property generator for GeoIDs - #2484
Conversation
…-disable-integrated-services-3
…-disable-integrated-services-3
gino-m
left a comment
There was a problem hiding this comment.
Please also update the PR description to something a bit more self-descriptive, e.g., "Implement property generator for GeoIDs"
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2484 +/- ##
=======================================
Coverage 62.13% 62.13%
=======================================
Files 121 121
Lines 2947 2947
Branches 441 441
=======================================
Hits 1831 1831
Misses 1053 1053
Partials 63 63 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
…-disable-integrated-services-3
…-disable-integrated-services-3
|
Doesn't WHISP already issue geoIDs? Why do we need a separate generator for GeoIDs? |
…-disable-integrated-services-3
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. |
…-disable-integrated-services-3
…-disable-integrated-services-3 # Conflicts: # functions/src/common/datastore.ts # functions/src/on-create-loi.ts
closes #2438
closes #1537
closes #2113