Fix missing geo calculations - Githubissues

fpdcc / ccfp-asset-dashboard

CCFP Asset Dashboard

0 stars 1 forks source link

Fix missing geo calculations #267

Closed smcalilly closed 10 months ago

smcalilly commented 10 months ago

Overview

closes #265

Includes a Heroku review app config change to support a different database connection (to be addressed in another PR, but related to #252)

Notes

This happened due to two things:

for the Zone model, the import was skipping the zones, so I changed it to an update_or_create
for districts, moving self.save() outside of the loop fixed this...dunno why

Testing Instructions

visit the review app: https://ccfp-asset-d-feature-di-1nlzy2.herokuapp.com/
navigate to a project with some assets (here's one). see how the projects don't include the districts/zones
add/remove an asset. visit the project overview page and see how the districts/zones get updated