easierdata / web3-geo-dashboard

1 stars 0 forks source link

Adjust cid enriched geojson so datetime and cid go inside properties #26

Closed jsolly closed 1 year ago

jsolly commented 1 year ago

Context

Currently the cid enriched geojson looks like this:

    "features": [
        {
            "type": "Feature",
            "properties": {
                "PATH": 13,
                "ROW": 27
            },
            "geometry": {
                "type": "Polygon",
                "coordinates": [
                    [
                        [-72.34003371649325, 46.874317393516314],
                    ]
                ]
            },
            "cid": "QmZ3hux2mwyVZz1nhsLTF8EorPHsHEzWExhdP4pge1q9mr",
            "datetime": "2022-10-17T15:35:43.937866Z"
        },
    ]

More correct version

        "features": [
            {
                "type": "Feature",
                "properties": {
                    "PATH": 13,
                    "ROW": 27,
                    "cid": "QmZ3hux2mwyVZz1nhsLTF8EorPHsHEzWExhdP4pge1q9mr",
                    "datetime": "2022-10-17T15:35:43.937866Z"
                },
                "geometry": {
                    "type": "Polygon",
                    "coordinates": [
                        [-72.34003371649325, 46.874317393516314],
                    ]
                }
            },
        ]
matthewnanas commented 1 year ago

Addressed in 019a8d7e5469809b3f289e79964175bcc0a9fc1e

jsolly commented 1 year ago

Hey looks good. Thx. I can start adjusting the popup based on these new properties.

Surely this meant https://github.com/easierdata/web3-geo-dashboard/blob/staging/data_processing/retrieve_cids.ts was changed along with the geojson? That commit you just shared only changed the result geojson, not any code that generates the geojson.

matthewnanas commented 1 year ago

Sorry about that, the retrieval script has been updated in my latest commit