Skip to content
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

Add option to export meter data in GeoJSON #3900

Merged
merged 4 commits into from
Mar 20, 2023

Conversation

nllong
Copy link
Member

@nllong nllong commented Mar 16, 2023

Any background context you want to provide?

Exporting Meter data isn't readily available. One can export from the "meters" tables within a single property.

What's this PR do?

Since meters and meter readings are relational, adding this export functionality to a flat CSV file doesn't make sense. We could eventually add to XLSX (similar to ESPM). This PR adds the data to a geojson record.

  • Adds meters and meter readings to the property obj in geojson
  • Adds option to exclude the export in the modal (the default is to exclude)

How should this be manually tested?

  • go to an org with properties that have meters
  • select a few properties
  • check the "export meter readings" and then "export geojson"
  • verify that the exported geojson file has the meter data.

What are the relevant tickets?

further address #2946

Screenshots (if appropriate)

image

image

@nllong nllong added the Feature Add this label to new features. This will be reflected in the change log when generated. label Mar 16, 2023
@nllong nllong requested a review from kflemin March 16, 2023 02:38
@axelstudios axelstudios merged commit 10043f3 into develop Mar 20, 2023
@axelstudios axelstudios deleted the add-option-to-export-meters-geojson branch March 20, 2023 00:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Add this label to new features. This will be reflected in the change log when generated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants