search

NYCPlanning / db-developments

🏠 🏘️ 🏗️ Developments Database
https://nycplanning.github.io/db-developments
8 stars 2 forks source link

update cdta sql to correctly generate table #499

Closed td928 closed 2 years ago

td928 commented 2 years ago

Overview

found the error causing the cdta table to be empty. The sql query insert the results into the wrong table name, probably due to copying over from another file but forgot to modify the name. Tested local output and should work properly now.

mbh329 commented 2 years ago

I personally can't test this to see if the table is now populated because the last PR wasn't merged in before creating this new feature branch

mbh329 commented 2 years ago

@td928 just make sure that the link for cdta makes it into the readme

td928 commented 2 years ago

yeah good comment by @mbh329 to add in the link in readme so to avoid another pull request for just that change. But unfortunately won't be able to test the link until this PR merged in and actions run to see if it really worked.